Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21472
Change subject: IMPALA-13111: Fix the calculation of fragment ids for impala-gdb.py ...................................................................... IMPALA-13111: Fix the calculation of fragment ids for impala-gdb.py The gdb helpers in impala-gdb.py provide functions to look on the stack for the information added in IMPALA-6416 and get the fragment/query ids. Right now, it is incorrectly using a signed integer, which leads to incorrect ids like this: -3cbda1606b3ade7c:f170c4bd00000000 This changes the logic to AND the integer with an 0xFF* sequence of the right length. This forces the integer to be unsigned, producing the right query id. Testing: - Ran this on a minidump and verified the the listed query ids were valid (and existed in the profile log) Change-Id: I59798407e99ee0e9100cac6b4b082cdb85ed43d1 --- M lib/python/impala_py_lib/gdb/impala-gdb.py 1 file changed, 8 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/72/21472/1 -- To view, visit http://gerrit.cloudera.org:8080/21472 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I59798407e99ee0e9100cac6b4b082cdb85ed43d1 Gerrit-Change-Number: 21472 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
