Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/19322 )
Change subject: IMPALA-9551: Allow mixed complex types in select list ...................................................................... Patch Set 13: Code-Review+1 (4 comments) http://gerrit.cloudera.org:8080/#/c/19322/12/be/src/runtime/complex-value-writer.h File be/src/runtime/complex-value-writer.h: http://gerrit.cloudera.org:8080/#/c/19322/12/be/src/runtime/complex-value-writer.h@48 PS12, Line 48: // out field names and types. This function can call itself recursively in case of > Done Done http://gerrit.cloudera.org:8080/#/c/19322/12/be/src/runtime/complex-value-writer.inline.h File be/src/runtime/complex-value-writer.inline.h: http://gerrit.cloudera.org:8080/#/c/19322/12/be/src/runtime/complex-value-writer.inline.h@226 PS12, Line 226: template <class JsonStream> > Removed this comment because this method is documented in the .h file. Done http://gerrit.cloudera.org:8080/#/c/19322/12/fe/src/main/java/org/apache/impala/analysis/InlineViewRef.java File fe/src/main/java/org/apache/impala/analysis/InlineViewRef.java: http://gerrit.cloudera.org:8080/#/c/19322/12/fe/src/main/java/org/apache/impala/analysis/InlineViewRef.java@404 PS12, Line 404: } > Good point, moved to putOneStructMemberIntoSmaps(). Done http://gerrit.cloudera.org:8080/#/c/19322/12/tests/query_test/test_nested_types.py File tests/query_test/test_nested_types.py: http://gerrit.cloudera.org:8080/#/c/19322/12/tests/query_test/test_nested_types.py@185 PS12, Line 185: if vector.get_value('exec_option')['disable_codegen'] == 'False': > Having struct types in the select list is not yet supported with codegen - A code comment about why it's disabled (mention the ticket) would be welcome. -- To view, visit http://gerrit.cloudera.org:8080/19322 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I476d98884b5fd192dfcd4feeec7947526aebe993 Gerrit-Change-Number: 19322 Gerrit-PatchSet: 13 Gerrit-Owner: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Peter Rozsa <pro...@cloudera.com> Gerrit-Comment-Date: Wed, 25 Jan 2023 16:54:12 +0000 Gerrit-HasComments: Yes
