impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dimitris Tsirogiannis (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5503: [DOCS] Document how to specify coordinator/executor nodes
Date Tue, 20 Jun 2017 22:45:45 GMT
Dimitris Tsirogiannis has posted comments on this change.

Change subject: IMPALA-5503: [DOCS] Document how to specify coordinator/executor nodes
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/7237/1/docs/topics/impala_scalability.xml
File docs/topics/impala_scalability.xml:

Line 358:         <cmdname>impala-shell</cmdname> or business intelligence tools.
It's good to explicitly mention that users can't connect to non-coordinator nodes using any
of the supported clients/protocols.


PS1, Line 373: for each host that you want to act
             :             exclusively as a coordinator.
I think it may be clearer if we say that "is_executor=false it means that this host is not
going to act as an executor". Similarly, for the is_coordinator flag.


-- 
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: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Mostafa Mokhtar <mmokhtar@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message