Alex Rodoni 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: (4 comments) http://gerrit.cloudera.org:8080/#/c/14860/1/docs/topics/impala_explain_plan.xml File docs/topics/impala_explain_plan.xml: http://gerrit.cloudera.org:8080/#/c/14860/1/docs/topics/impala_explain_plan.xml@147 PS1, Line 147: > : When the <codeph>MT_DOP</codeph> query option is set to a value larger than : <codeph>0</codeph>, the <codeph>#Inst</codeph> column in the output shows the number of : fragment instances. Impala decomposes each query into smaller units of wor > The #Inst column in the output will be also shown when MT_DOP=0, with the s Done http://gerrit.cloudera.org:8080/#/c/14860/1/docs/topics/impala_explain_plan.xml@168 PS1, Line 168: > This and the zeros below should be ones. Done 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 Done 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 Done -- 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-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Mon, 09 Dec 2019 19:03:25 +0000 Gerrit-HasComments: Yes
