Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8865 )

Change subject: IMPALA-2181: [DOCS] Document changes to SET output
......................................................................


Patch Set 3: Code-Review+2

(1 comment)

Looks good, just had one optional suggestion

http://gerrit.cloudera.org:8080/#/c/8865/3/docs/topics/impala_set.xml
File docs/topics/impala_set.xml:

http://gerrit.cloudera.org:8080/#/c/8865/3/docs/topics/impala_set.xml@69
PS3, Line 69:             related to internal development of Impala or features 
that are not yet finalized.
Is it worth explicitly saying something like "Development query options may be 
modified or removed without notice."



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iade7cb326715ebbb8518230d518d05601d615f61
Gerrit-Change-Number: 8865
Gerrit-PatchSet: 3
Gerrit-Owner: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Gabor Kaszab <gaborkas...@cloudera.com>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Fri, 12 Jan 2018 16:08:17 +0000
Gerrit-HasComments: Yes

Reply via email to