Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5120: Default to partitioned join when stats are missing ......................................................................
IMPALA-5120: Default to partitioned join when stats are missing Previously, we defaulted to broadcast join when stats were missing, but this can lead to disastrous plans when the right hand side is actually large. Its always difficult to make good plans when stats are missing, but defaulting to partitioned joins should reduce the risk of disastrous plans. Testing: - Added a planner test that joins a table with no stats. Change-Id: Ie168ecfcd5e7c5d3c60d16926c151f8f134c81e0 Reviewed-on: http://gerrit.cloudera.org:8080/6803 Reviewed-by: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Tested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/planner/DistributedPlanner.java M testdata/workloads/functional-planner/queries/PlannerTest/joins.test 2 files changed, 24 insertions(+), 1 deletion(-) Approvals: Impala Public Jenkins: Verified Thomas Tauber-Marshall: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/6803 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie168ecfcd5e7c5d3c60d16926c151f8f134c81e0 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
