Norbert Luksa has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14860 )

Change subject: IMPALA-9207: [DOCS] Documented the #Inst in exec summary
......................................................................


Patch Set 2: Code-Review+1

(2 comments)

Found some more nits, otherwise lgtm.

http://gerrit.cloudera.org:8080/#/c/14860/2/docs/topics/impala_explain_plan.xml
File docs/topics/impala_explain_plan.xml:

http://gerrit.cloudera.org:8080/#/c/14860/2/docs/topics/impala_explain_plan.xml@175
PS2, Line 175:
nit: extra space


http://gerrit.cloudera.org:8080/#/c/14860/2/docs/topics/impala_mem_limit.xml
File docs/topics/impala_mem_limit.xml:

http://gerrit.cloudera.org:8080/#/c/14860/2/docs/topics/impala_mem_limit.xml@209
PS2, Line 209: #Hosts
nit: #Inst



--
To view, visit http://gerrit.cloudera.org:8080/14860
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I938930c66144ba6bce766981d363abe4b28ba524
Gerrit-Change-Number: 14860
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Norbert Luksa <[email protected]>
Gerrit-Comment-Date: Sun, 08 Dec 2019 13:57:36 +0000
Gerrit-HasComments: Yes

Reply via email to