Bikramjeet Vig has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17954 )
Change subject: IMPALA-10973: Do not schedule empty scan nodes to coordinator ...................................................................... IMPALA-10973: Do not schedule empty scan nodes to coordinator Until now fragments with scan nodes that have no scan ranges were scheduled to the coordinator, even if it is an exclusive coordinator. This could possibly lead to a lot of work to be scheduled to the coordinator. This patch changes the logic to choose a random executor instead. Change-Id: Ie31df3861aad2e3e91cab621ff122a4f721905ef Reviewed-on: http://gerrit.cloudera.org:8080/17954 Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Reviewed-by: Abhishek Rawat <ara...@cloudera.com> Reviewed-by: Bikramjeet Vig <bikramjeet....@cloudera.com> --- M be/src/scheduling/scheduler.cc M be/src/scheduling/scheduler.h 2 files changed, 17 insertions(+), 14 deletions(-) Approvals: Impala Public Jenkins: Verified Abhishek Rawat: Looks good to me, but someone else must approve Bikramjeet Vig: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17954 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie31df3861aad2e3e91cab621ff122a4f721905ef Gerrit-Change-Number: 17954 Gerrit-PatchSet: 3 Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>