Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24902 )
Change subject: IMPALA-15385: Make AVG_COLLECTION_SIZE a query option ...................................................................... IMPALA-15385: Make AVG_COLLECTION_SIZE a query option AVG_COLLECTION_SIZE affects the cardinalty estimation of some plan nodes, which in turn affects the memory estimation. This patch changes AVG_COLLECTION_SIZE from a hard-coded constant to a query option so that user can set it for each query. Testing: - Added a new PlannerTest. Change-Id: I518f76b38ba926e6ece196f6e90c60f1fa15b5ff Reviewed-on: http://gerrit.cloudera.org:8080/24902 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/service/query-options.cc M be/src/service/query-options.h M common/thrift/ImpalaService.thrift M common/thrift/Query.thrift M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java M fe/src/main/java/org/apache/impala/planner/PlannerContext.java M fe/src/main/java/org/apache/impala/planner/UnnestNode.java M fe/src/test/java/org/apache/impala/planner/PlannerTest.java A testdata/workloads/functional-planner/queries/PlannerTest/avg-collection-size-non-zero.test A testdata/workloads/functional-planner/queries/PlannerTest/avg-collection-size.test 11 files changed, 122 insertions(+), 16 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24902 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I518f76b38ba926e6ece196f6e90c60f1fa15b5ff Gerrit-Change-Number: 24902 Gerrit-PatchSet: 4 Gerrit-Owner: Xuebin Su <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Nandor Kollar <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Xuebin Su <[email protected]>
