You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2020/11/24 17:26:30 UTC
[Impala-ASF-CR] IMPALA-10351,IMPALA-9812: enable mt dop for DML by default
Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16775
to look at the new patch set (#3).
Change subject: IMPALA-10351,IMPALA-9812: enable mt_dop for DML by default
......................................................................
IMPALA-10351,IMPALA-9812: enable mt_dop for DML by default
This allows setting mt_dop for any query with any configuration.
Before this patch it was not supported for DML.
--unlock_mt_dop and --mt_dop_auto_fallback are now ignored.
Testing:
* Updated tests to reflect new behaviour.
* Removed irrelevant tests for fallback/validation.
* Ran exhaustive tests.
Change-Id: I66331481260fe4b69d9e95b0200029b14d230ade
---
M be/src/common/global-flags.cc
M be/src/util/backend-gflag-util.cc
M bin/run-all-tests.sh
M common/thrift/BackendGflags.thrift
M fe/src/main/java/org/apache/impala/common/RuntimeEnv.java
M fe/src/main/java/org/apache/impala/planner/Planner.java
M fe/src/main/java/org/apache/impala/service/BackendConfig.java
M fe/src/test/java/org/apache/impala/planner/PlannerTest.java
M testdata/workloads/functional-planner/queries/PlannerTest/mt-dop-validation.test
D testdata/workloads/functional-query/queries/QueryTest/mt-dop-auto-fallback.test
M tests/custom_cluster/test_mt_dop.py
11 files changed, 69 insertions(+), 132 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/75/16775/3
--
To view, visit http://gerrit.cloudera.org:8080/16775
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I66331481260fe4b69d9e95b0200029b14d230ade
Gerrit-Change-Number: 16775
Gerrit-PatchSet: 3
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>