John Russell has posted comments on this change. Change subject: IMPALA-5503: [DOCS] Document how to specify coordinator/executor nodes ......................................................................
Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7237/1/docs/topics/impala_scalability.xml File docs/topics/impala_scalability.xml: Line 358: Then, you specify that the > It's good to explicitly mention that users can't connect to non-coordinator Done. I'll split this paragraph into 2 so it's clear which set of hosts is under discussion for each point. PS1, Line 373: : <ul> > I think it may be clearer if we say that "is_executor=false it means that t Done. I'll give that a try. Generally it's considered more likely to confuse people to introduce multiple "negative" ideas in succession. E.g. "parameter=false mean don't do X". So we may need to fine-tune the wording some more. -- To view, visit http://gerrit.cloudera.org:8080/7237 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia20db6af212122b1f87fc6999f8683860beb2bad Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Mostafa Mokhtar <[email protected]> Gerrit-HasComments: Yes
