Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14707 )
Change subject: IMPALA-9081: fix mt_dop validation tests ...................................................................... IMPALA-9081: fix mt_dop validation tests This undoes the hack of pretending that it's not a test environment for that single test. That had side effects, e.g. for the metadata loading path. Instead we have a special flag to enable the validation code in frontend tests. Note that the plans change to include join build sinks as an expected result of undoing the hack. Change-Id: I2e8823c562395e13f318d1ad6eed883d2d9d771f Reviewed-on: http://gerrit.cloudera.org:8080/14707 Reviewed-by: Anurag Mantripragada <[email protected]> Reviewed-by: Thomas Tauber-Marshall <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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/test/java/org/apache/impala/planner/PlannerTest.java M testdata/workloads/functional-planner/queries/PlannerTest/mt-dop-validation-hdfs-num-rows-est-enabled.test M testdata/workloads/functional-planner/queries/PlannerTest/mt-dop-validation.test 5 files changed, 87 insertions(+), 52 deletions(-) Approvals: Anurag Mantripragada: Looks good to me, but someone else must approve Thomas Tauber-Marshall: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14707 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2e8823c562395e13f318d1ad6eed883d2d9d771f Gerrit-Change-Number: 14707 Gerrit-PatchSet: 6 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Anurag Mantripragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
