Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/18099 )
Change subject: IMPALA-11049: Substitute order by elements when creating SortInfo ...................................................................... Patch Set 3: Code-Review+1 (1 comment) Thanks for applying the changes. LGTM! http://gerrit.cloudera.org:8080/#/c/18099/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18099/1//COMMIT_MSG@15 PS1, Line 15: ts_' in-place, and : use these substituted exprs to create 'SortInfo'. B > Done I mean the EXPLAIN output already prints out the rewritten SQL, even without this patch. So maybe we don't need this sentence (and neither the new examples)? Unless something significant changed in the EXPLAIN output, but AFAIK this is not the case here. -- To view, visit http://gerrit.cloudera.org:8080/18099 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2780e04a6d5a32e224cd0470cf6f166a832363ec Gerrit-Change-Number: 18099 Gerrit-PatchSet: 3 Gerrit-Owner: wangsheng <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: wangsheng <[email protected]> Gerrit-Comment-Date: Fri, 17 Dec 2021 11:10:24 +0000 Gerrit-HasComments: Yes
