Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11955 )
Change subject: IMPALA-7844: HAVING clause cannot support ordinals ...................................................................... Patch Set 2: Change mostly passed the pre-commit test: https://jenkins.impala.io/job/ubuntu-16.04-from-scratch/3640/console A query failed due to a memory-related error, but my speculation is that the error is spurious and not related to this change. Forbidding ordinals should either work (for queries that don't use them), or plainly fail (if we had any that did, as in the FE tests.) -- To view, visit http://gerrit.cloudera.org:8080/11955 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic2b9f9e8c60fe2b25e20c57c2ffc31d8e59d5861 Gerrit-Change-Number: 11955 Gerrit-PatchSet: 2 Gerrit-Owner: Paul Rogers <par0...@yahoo.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Paul Rogers <par0...@yahoo.com> Gerrit-Comment-Date: Wed, 21 Nov 2018 02:37:24 +0000 Gerrit-HasComments: No