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 3:

(2 comments)

Thanks for working on this!

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_shutdown.xml?


http://gerrit.cloudera.org:8080/#/c/23031/3/docs/topics/impala_kill_query.xml@44
PS3, Line 44: query_id
How about adding a <varname> tag, as in docs/topics/impala_shutdown.xml?



--
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: 3
Gerrit-Owner: Sanjana Malhotra <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Xuebin Su <[email protected]>
Gerrit-Comment-Date: Thu, 19 Jun 2025 07:03:46 +0000
Gerrit-HasComments: Yes

Reply via email to