Jason Fehr has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21074 )

Change subject: IMPALA-12602: Unregister queries on idle timeout
......................................................................


Patch Set 7:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/21074/7/be/src/service/impala-server.cc
File be/src/service/impala-server.cc:

http://gerrit.cloudera.org:8080/#/c/21074/7/be/src/service/impala-server.cc@2863
PS7, Line 2863: status
nit: mark const


http://gerrit.cloudera.org:8080/#/c/21074/7/docs/topics/impala_timeouts.xml
File docs/topics/impala_timeouts.xml:

http://gerrit.cloudera.org:8080/#/c/21074/7/docs/topics/impala_timeouts.xml@130
PS7, Line 130:             exception details.
Please add an explanation that the status will be INACTIVE_QUERY_EXPIRED if the 
query was in a good state at expiration or will be the error status if the 
query was not in a good state at expiration.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iacfc285ed3587892c7ec6f7df3b5f71c9e41baf0
Gerrit-Change-Number: 21074
Gerrit-PatchSet: 7
Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Comment-Date: Wed, 20 Mar 2024 21:54:57 +0000
Gerrit-HasComments: Yes

Reply via email to