You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Abhishek Sharma (Code Review)" <ge...@cloudera.org> on 2018/04/24 21:02:53 UTC

[Impala-ASF-CR] IMPALA-5392: Added all stack frames to ThreadInfo summary.

Abhishek Sharma has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/10145 )

Change subject: IMPALA-5392: Added all stack frames to ThreadInfo summary.
......................................................................

IMPALA-5392: Added all stack frames to ThreadInfo summary.

The current implementation uses ThreadInfo.toString.
This implementation restricts the number of stack frames to 8.
As a part of this fix, only this particular constraint is removed.
Now all stack frames are included in the summary.
No other detail is added or edited or omitted apart from
the constraint stated above.

A standard ThreadInfo summary includes -
- Thread ID
- Thread Name
- Thread State
- Entire Stack Trace (if any)
- Monitor Info (if any)
- Lock Info

Change-Id: I80ab4aad03e0c1f01fecad6b87779531244c28b7
---
M fe/src/main/java/org/apache/impala/common/JniUtil.java
1 file changed, 76 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/45/10145/4
-- 
To view, visit http://gerrit.cloudera.org:8080/10145
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I80ab4aad03e0c1f01fecad6b87779531244c28b7
Gerrit-Change-Number: 10145
Gerrit-PatchSet: 4
Gerrit-Owner: Abhishek Sharma <ab...@gmail.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Charles Agnello <ch...@gmail.com>
Gerrit-Reviewer: Jim Apple <jb...@apache.org>