Hello Aman Sinha, Joe McDonnell, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24074
to look at the new patch set (#4).
Change subject: IMPALA-14482: Calcite planner: order by "random()" fixed
......................................................................
IMPALA-14482: Calcite planner: order by "random()" fixed
The SortRemoveConstantKeysRule in Calcite was removing functions
that did not have any column expressions in it.
This commit ensure that the isDeterministic method for the operator
returns the correct value.
The test for this can be found in:
test_sort.py::TestRandomSort::test_order_by_random
Change-Id: I6ebf3aa11030d98b447d08d1364f5f65eb2ca661
---
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/operators/ImpalaOperator.java
1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/74/24074/4
--
To view, visit http://gerrit.cloudera.org:8080/24074
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6ebf3aa11030d98b447d08d1364f5f65eb2ca661
Gerrit-Change-Number: 24074
Gerrit-PatchSet: 4
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
