Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16910 )
Change subject: [WIP] IMPALA-10410: Query hints for turn on/off UTF-8 behavior ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/16910/5/fe/src/main/java/org/apache/impala/analysis/Analyzer.java File fe/src/main/java/org/apache/impala/analysis/Analyzer.java: http://gerrit.cloudera.org:8080/#/c/16910/5/fe/src/main/java/org/apache/impala/analysis/Analyzer.java@2815 PS5, Line 2815: public void setIsUtf8Mode(boolean isUtf8Mode) throws AnalysisException{ I mentioned this to you out-of-band but thought I should mention that I think this will get lost on some expressions when they are substituted and re-analyzed in a different query block or with the root analyzer during planning. -- To view, visit http://gerrit.cloudera.org:8080/16910 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7fa20b62b5cb06169048b0785b70e85a9f21bf07 Gerrit-Change-Number: 16910 Gerrit-PatchSet: 5 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Fri, 08 Jan 2021 17:45:18 +0000 Gerrit-HasComments: Yes
