HADOOP-13434 inadvertently undid the fix made in HADOOP-12441.
The use of the bash builtin for kill was intentional, so let's restore that behavior.
Add quoting to Shell class
Fix kill command behavior under some Linux distributions.
SUCCESS: Integrated in Hadoop-trunk-Commit #10216 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10216/)
HADOOP-13467. Shell#getSignalKillCommand should use the bash builtin on (arp: rev b8ca84ac99d2cb2bfe43930eb9b178ee010e4812)
By the way I addressed the remaining two checkstyle indentation warnings on commit (checkstyle seems to want a different convention than we use in Hadoop). The whitespace warnings looked spurious too.
Thank you again Larry. I've committed this for 2.8.0.
No fix is necessary on 2.7.x.
I have verified that this patch returns the original intent and adds the quoting of the pid.
This message was automatically generated.
v2 patch with checkstyle fixes.
cc Vinod Kumar Vavilapalli, Wangda Tan.
I'll post a patch shortly.