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

Change subject: IMPALA-13844: Add /stacks web page to dump native thread stacks
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

LGTM. Carrying Michael's +1. Thanks for adding this useful feature!

http://gerrit.cloudera.org:8080/#/c/23652/3/be/src/common/init.cc
File be/src/common/init.cc:

http://gerrit.cloudera.org:8080/#/c/23652/3/be/src/common/init.cc@594
PS3, Line 594:   // We use SIGRTMIN+10 instead of SIGUSR1/SIGUSR2 because:
> The signal handler is not meant to be manually triggered. It's designed to
Ack



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9171c89d1d36726d98a4a61ca040d69254f50e92
Gerrit-Change-Number: 23652
Gerrit-PatchSet: 4
Gerrit-Owner: Arnab Karmakar <[email protected]>
Gerrit-Reviewer: Arnab Karmakar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Comment-Date: Sun, 28 Dec 2025 03:11:57 +0000
Gerrit-HasComments: Yes

Reply via email to