Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5641: mem-estimate should never be less than mem-reservation ......................................................................
IMPALA-5641: mem-estimate should never be less than mem-reservation This is implemented in the ResourceProfileBuilder to avoid duplicating the login in every plan node. Testing: Updated planner tests. Change-Id: I1e2853300371e31b13d81a763dbafb21709b16c4 Reviewed-on: http://gerrit.cloudera.org:8080/7703 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/planner/ResourceProfile.java M testdata/workloads/functional-planner/queries/PlannerTest/constant-folding.test M testdata/workloads/functional-planner/queries/PlannerTest/disable-codegen.test M testdata/workloads/functional-planner/queries/PlannerTest/fk-pk-join-detection.test M testdata/workloads/functional-planner/queries/PlannerTest/mt-dop-validation.test M testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test M testdata/workloads/functional-planner/queries/PlannerTest/sort-expr-materialization.test M testdata/workloads/functional-planner/queries/PlannerTest/spillable-buffer-sizing.test M testdata/workloads/functional-planner/queries/PlannerTest/tablesample.test 9 files changed, 193 insertions(+), 191 deletions(-) Approvals: Impala Public Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7703 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e2853300371e31b13d81a763dbafb21709b16c4 Gerrit-PatchSet: 7 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
