Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/8447 )
Change subject: IMPALA-2181: Add query option levels for display ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/8447/8/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/8447/8/be/src/service/impala-server.cc@1215 PS8, Line 1215: string_map["support_start_over"] = "false"; This may not be related to this change, but it would be good to think a bit about support_start_over option. Unlike other options, it is not an integer, and it should not be changed. It is read by Hue ccording to https://www.cloudera.com/documentation/enterprise/5-4-x/topics/impala_support_start_over.html I do not know how Hue accesses query options, but if via the output of "set;" hiding it by default may actually change the behavior of Hue. Even if this is not an issue, it would be probably better to separate this option clearly e.g. by giving it a separate level like "capability" and treat these query options as read only. ( this comment added support_start_over a long time ago: https://github.com/cloudera/Impala/commit/94a5ed487e04b7d2c9bf6bee399f0f464f289211b ) -- To view, visit http://gerrit.cloudera.org:8080/8447 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I75720d0d454527e1a0ed19bb43cf9e4f018ce1d1 Gerrit-Change-Number: 8447 Gerrit-PatchSet: 8 Gerrit-Owner: Gabor Kaszab <gaborkas...@cloudera.com> Gerrit-Reviewer: Attila Jeges <atti...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Gabor Kaszab <gaborkas...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com> Gerrit-Comment-Date: Tue, 07 Nov 2017 23:07:55 +0000 Gerrit-HasComments: Yes