Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24118 )

Change subject: IMPALA-14843: Show cancelled nodes in ExecSummary
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/24118/1/be/src/runtime/coordinator.cc
File be/src/runtime/coordinator.cc:

http://gerrit.cloudera.org:8080/#/c/24118/1/be/src/runtime/coordinator.cc@1802
PS1, Line 1802:
> nit: instead of a magic number, we could define a constant somewhere? Or me
Done. Extracted SINK_ID to be used here.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd1bf51112e7c8c5609bb9e9525d960efc524215
Gerrit-Change-Number: 24118
Gerrit-PatchSet: 3
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Comment-Date: Tue, 24 Mar 2026 08:34:23 +0000
Gerrit-HasComments: Yes

Reply via email to