Hello Philip Zeyliger, Kim Jin Chul,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/10347

to look at the new patch set (#2).

Change subject: Fix diagnostics path to not include the parent dir structure
......................................................................

Fix diagnostics path to not include the parent dir structure

Without the fix, the diagnostics tar file included the entire
directory structure of the diagnostics root dir.

Before:
=======
$ tar tf /tmp/impala-diagnostics-2018-05-08-11-59-39-spv8Eh.tar.gz
tmp/impala-diagnostics-2018-05-08-11-59-39-spv8Eh/
tmp/impala-diagnostics-2018-05-08-11-59-39-spv8Eh/stacks/
tmp/impala-diagnostics-2018-05-08-11-59-39-spv8Eh/stacks/jstack-0.txt
....

After:
=====
$ tar tf /tmp/impala-diagnostics-2018-05-08-12-01-51-Y0nlQI.tar.gz
impala-diagnostics-2018-05-08-12-01-51-Y0nlQI/
impala-diagnostics-2018-05-08-12-01-51-Y0nlQI/stacks/
impala-diagnostics-2018-05-08-12-01-51-Y0nlQI/stacks/jstack-0.txt
.....

Tested with python 2.6

Change-Id: I540f6c228a0315780d45cf11961f124478b5dd0c
---
M bin/diagnostics/collect_diagnostics.py
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/47/10347/2
--
To view, visit http://gerrit.cloudera.org:8080/10347
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I540f6c228a0315780d45cf11961f124478b5dd0c
Gerrit-Change-Number: 10347
Gerrit-PatchSet: 2
Gerrit-Owner: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Kim Jin Chul <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>

Reply via email to