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 <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Mon, 16 Apr 2018 18:22:40 +0000 Gerrit-HasComments: Yes
