Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22826 )

Change subject: IMPALA-14000: Dump jstacks first in dump-stacktraces.sh
......................................................................


Patch Set 1:

(1 comment)

In addition to the change in logic, we might be able to make the gdb piece more 
fault tolerant.

http://gerrit.cloudera.org:8080/#/c/22826/1/bin/dump-stacktraces.sh
File bin/dump-stacktraces.sh:

http://gerrit.cloudera.org:8080/#/c/22826/1/bin/dump-stacktraces.sh@30
PS1, Line 30: thread apply all bt
thread apply all has a "-c" option to continue past an error. Here is the 
description from "help thread apply all":

By default, an error raised during the execution of COMMAND
aborts "thread apply".
...
  -c
    Print any error raised by COMMAND and continue.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8f610ee4d4934fe950a9f56cf74a7e76e5d63651
Gerrit-Change-Number: 22826
Gerrit-PatchSet: 1
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Tue, 06 May 2025 01:45:44 +0000
Gerrit-HasComments: Yes

Reply via email to