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/24691
to look at the new patch set (#4).
Change subject: IMPALA-15261: Calcite Planner: Fix broken tests due to plan
changes.
......................................................................
IMPALA-15261: Calcite Planner: Fix broken tests due to plan changes.
The commit IMPALA-15195 caused some significant plan changes for the
Calcite planner. As a result, some tests were broken when running
with the Calcite planner on.
In mt-dop-parquet-scheduling.test: Some node numbers were changed
In test_processing_cost.py: Plan changes caused this to fail. Since
this is a backend test, these failures are ok, but should be fixed
at some point. IMPALA-15260 has been filed for this.
In test_executor_groups: This is also a backend test. A Jira was
already filed for the memory estimate issue, IMPALA-14816. A comment
was added to this Jira to handle this problem too.
In test_iceberg.py: Fixed this by running with "straight_join", since
this shouldn't affect the integrity of the test.
Change-Id: I77683c0ab2f506b6c638255b63bde243b33dd8fb
---
M
testdata/workloads/functional-query/queries/QueryTest/mt-dop-parquet-scheduling.test
M tests/common/skip.py
M tests/custom_cluster/test_executor_groups.py
M tests/query_test/test_iceberg.py
M tests/query_test/test_processing_cost.py
5 files changed, 47 insertions(+), 40 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/91/24691/4
--
To view, visit http://gerrit.cloudera.org:8080/24691
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I77683c0ab2f506b6c638255b63bde243b33dd8fb
Gerrit-Change-Number: 24691
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]>
