Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13001 )
Change subject: Fix the stale comment on ImpalaServer::RegisterQuery header ...................................................................... Fix the stale comment on ImpalaServer::RegisterQuery header Prior to this change, the comment from ImpalaServer::RegisterQuery asserts: "Registers the query exec state with client_request_state_map_ using the globally unique query_id and add the query id to session state's open query list." This comment is partially inaccurate about the current designs of this function since it does not add the query_id to the session state's open query list. Change-Id: If743e718f7322929acb710b36f85978dfbe981bf Reviewed-on: http://gerrit.cloudera.org:8080/13001 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/service/impala-server.h 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/13001 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If743e718f7322929acb710b36f85978dfbe981bf Gerrit-Change-Number: 13001 Gerrit-PatchSet: 3 Gerrit-Owner: Vincent Tran <vtt...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>