Aman Sinha has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15807
Change subject: IMPALA-9539: Enable CNF rewrites by default ...................................................................... IMPALA-9539: Enable CNF rewrites by default This patch enables the conjunctive normal form rewrites by default by setting enable_cnf_rewrites to true. Functional and performance testing with this flag set to true did not uncover any regressions and showed significant performance gains for queries with disjunctions in the tpch and tpcds suites. Testing: - Updated the PlannerTest tests with plan changes in various test suites. Removed previously added tpch tests which were explicitly setting this flag to true. - I had previously added a test in convert-to-cnf.test with enable_cnf_rewrites=false, so I did not add any new tests with this flag disabled. Change-Id: I4dde86e092c61d71ddf9081f768072ced470b589 --- M common/thrift/ImpalaInternalService.thrift M fe/src/test/java/org/apache/impala/planner/PlannerTest.java M testdata/workloads/functional-planner/queries/PlannerTest/constant-folding.test M testdata/workloads/functional-planner/queries/PlannerTest/tpcds-all.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-all.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-kudu.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-nested.test M testdata/workloads/functional-planner/queries/PlannerTest/tpch-views.test 8 files changed, 499 insertions(+), 660 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/07/15807/1 -- To view, visit http://gerrit.cloudera.org:8080/15807 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I4dde86e092c61d71ddf9081f768072ced470b589 Gerrit-Change-Number: 15807 Gerrit-PatchSet: 1 Gerrit-Owner: Aman Sinha <[email protected]>
