Xuebin Su has posted comments on this change. ( http://gerrit.cloudera.org:8080/23031 )
Change subject: IMPALA-12648: [DOCS] Documentation for Kill Query statements ...................................................................... Patch Set 4: (3 comments) > Uploaded patch set 4. Thanks! http://gerrit.cloudera.org:8080/#/c/23031/3/docs/topics/impala_kill_query.xml File docs/topics/impala_kill_query.xml: http://gerrit.cloudera.org:8080/#/c/23031/3/docs/topics/impala_kill_query.xml@24 PS3, Line 24: KILL QUERY > How about changing it to all upper-case letters, as in docs/topics/impala_s Done http://gerrit.cloudera.org:8080/#/c/23031/3/docs/topics/impala_kill_query.xml@44 PS3, Line 44: lt;'<var > How about adding a <varname> tag, as in docs/topics/impala_shutdown.xml? Done http://gerrit.cloudera.org:8080/#/c/23031/4/docs/topics/impala_kill_query.xml File docs/topics/impala_kill_query.xml: http://gerrit.cloudera.org:8080/#/c/23031/4/docs/topics/impala_kill_query.xml@44 PS4, Line 44: < Maybe we don't need to add < and > here. What do you think? Thanks! -- To view, visit http://gerrit.cloudera.org:8080/23031 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I51efbdeb585bad358b3e44ea4f555f62bfee4f00 Gerrit-Change-Number: 23031 Gerrit-PatchSet: 4 Gerrit-Owner: Sanjana Malhotra <malhotrasanjana1...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Xuebin Su <x...@cloudera.com> Gerrit-Comment-Date: Mon, 23 Jun 2025 06:28:51 +0000 Gerrit-HasComments: Yes
