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

Change subject: IMPALA-9196: Dump jstack and collect logs when tests timeout
......................................................................


Patch Set 2:

I'm uncertain about how the privileges work. There are ptrace limitations in 
Ubuntu that restrict ptrace by the same user to a parent process, which I think 
is why the gdb part of this script works. I'm not sure what permissions jstack 
would need, and if this would work.

If you haven't already, a test that you could run for the permissions is to run 
the end to end tests and set TIMEOUT_FOR_RUN_ALL_TESTS_MINS to some modest 
value (15 mins) and verify you get the logs you want and jstack works.

Once we verify that the permissions are ok in the normal way we run this, the 
code looks good to me.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8a5b140024c236209c7e44149660189890b9d06
Gerrit-Change-Number: 14794
Gerrit-PatchSet: 2
Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Comment-Date: Tue, 26 Nov 2019 01:32:20 +0000
Gerrit-HasComments: No

Reply via email to