Dan Hecht has posted comments on this change. Change subject: IMPALA-5147: Add the ability to exclude hosts from query execution ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6628/1/be/src/scheduling/scheduler.cc File be/src/scheduling/scheduler.cc: Line 207: current_membership_.insert(make_pair(item.key, be_desc)); > I am a little confused with the last part of your comment: "That way, e.g. Hue would like to be able to load balance across the set of coordinators. Before the is-coordinator change, they could do that by looking at the membership list (since all impalads acted as coordinators). Hue still needs a way to find the coordinators, and if you remove them from the membership set, they won't be able to use that. Instead the proposal is to continue having all impalads be part of the "cluster membership", and instead, augment the values with info about whether the node is a coordinator and/or executor. -- To view, visit http://gerrit.cloudera.org:8080/6628 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d2ff7f341c9d2b0649e4d14561077e166ad7c4d Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-HasComments: Yes
