You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@livy.apache.org by js...@apache.org on 2017/07/03 06:51:36 UTC
incubator-livy git commit: LIVY-361. If LIVY_PID_DIR contains space,
stop livy-server failed
Repository: incubator-livy
Updated Branches:
refs/heads/master 424776128 -> 26d69a496
LIVY-361. If LIVY_PID_DIR contains space, stop livy-server failed
[https://issues.cloudera.org/browse/LIVY-361](https://issues.cloudera.org/browse/LIVY-361)
There is an error in cheking pid file existence while stop livy-server. If LIVY_PID_DIR contains space, It is unable to get the pid file within it, so stop livy-server failed.
Author: davidxdh <xu...@zte.com.cn>
Closes #8 from davidxdh/dev361.
Project: http://git-wip-us.apache.org/repos/asf/incubator-livy/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-livy/commit/26d69a49
Tree: http://git-wip-us.apache.org/repos/asf/incubator-livy/tree/26d69a49
Diff: http://git-wip-us.apache.org/repos/asf/incubator-livy/diff/26d69a49
Branch: refs/heads/master
Commit: 26d69a496415bb743af65dc3dccd12a667660bcc
Parents: 4247761
Author: davidxdh <xu...@zte.com.cn>
Authored: Mon Jul 3 14:51:30 2017 +0800
Committer: jerryshao <ss...@hortonworks.com>
Committed: Mon Jul 3 14:51:30 2017 +0800
----------------------------------------------------------------------
bin/livy-server | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-livy/blob/26d69a49/bin/livy-server
----------------------------------------------------------------------
diff --git a/bin/livy-server b/bin/livy-server
index 16ee4fd..79159f4 100755
--- a/bin/livy-server
+++ b/bin/livy-server
@@ -144,7 +144,7 @@ case $option in
;;
(stop)
- if [ -f $pid ]; then
+ if [ -f "$pid" ]; then
TARGET_ID="$(cat "$pid")"
if [[ $(ps -p "$TARGET_ID" -o comm=) =~ "java" ]]; then
echo "stopping livy-server"