Vuk Ercegovac has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10070 )

Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10070/3/docs/topics/impala_compute_stats.xml
File docs/topics/impala_compute_stats.xml:

http://gerrit.cloudera.org:8080/#/c/10070/3/docs/topics/impala_compute_stats.xml@53
PS3, Line 53: column_list
this is a non-terminal; I see the verbal expansion on L81-83. If you typically 
expand this more formally (as done for example on L59), pls do so for this as 
well for consistency.
does it make sense to have an example, since it can be a large resource saver 
for wide tables, large datatypes?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I609c38eac29e36eca008bfb66f5e78f5491e719a
Gerrit-Change-Number: 10070
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
Gerrit-Comment-Date: Mon, 16 Apr 2018 18:22:40 +0000
Gerrit-HasComments: Yes

Reply via email to