Hello Bharath Vissapragada, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11178
to look at the new patch set (#2).
Change subject: Make collect_diagnostics executable and minor argument-parsing
changes.
......................................................................
Make collect_diagnostics executable and minor argument-parsing changes.
collect_diagnostics.py was missing a shebang (#!) at its
beginning as well as the executable bit. Since it's
meant to be runnable standalone, I've added those.
I was able to also simplify the argument handling around
the single required --pid argument.
Change-Id: If4d021c2f6f9dec62d6865d32ec0419e41a2441c
---
M bin/diagnostics/collect_diagnostics.py
1 file changed, 9 insertions(+), 10 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/11178/2
--
To view, visit http://gerrit.cloudera.org:8080/11178
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If4d021c2f6f9dec62d6865d32ec0419e41a2441c
Gerrit-Change-Number: 11178
Gerrit-PatchSet: 2
Gerrit-Owner: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>