You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2021/07/08 18:19:35 UTC

Change in asterixdb[master]: [ASTERIXDB-2922][FUN] Fix get_type function bug w/ multiple records.

From Glenn Galvizo <gg...@uci.edu>:

Hello Anon. E. Moose #1000171, Wail Alkowaileet, Jenkins, Wael Alkowaileet, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12203

to look at the new patch set (#3).

Change subject: [ASTERIXDB-2922][FUN] Fix get_type function bug w/ multiple records.
......................................................................

[ASTERIXDB-2922][FUN] Fix get_type function bug w/ multiple records.

- user mode changes: no
- storage format changes: no
- interface changes: no

Previously, get_type used a string builder that would retain the results
of all past records. We now use the raw string from the type instance
itself.

Change-Id: I9564bcdeb94dd141a03cc12a2f7a61f1e57d250b
---
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/gettype/gettype.2.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/gettype/gettype.3.ddl.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/gettype/gettype.4.query.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/types/gettype/gettype.2.adm
A asterixdb/asterix-app/src/test/resources/runtimets/results/types/gettype/gettype.3.adm
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/GetTypeDescriptor.java
6 files changed, 153 insertions(+), 8 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/03/12203/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12203
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I9564bcdeb94dd141a03cc12a2f7a61f1e57d250b
Gerrit-Change-Number: 12203
Gerrit-PatchSet: 3
Gerrit-Owner: Glenn Galvizo <gg...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo <gg...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Wael Alkowaileet <wa...@gmail.com>
Gerrit-Reviewer: Wail Alkowaileet
Gerrit-MessageType: newpatchset