commit 63e6bdcd48c787341dfa1a02314b53c08309bcc5
author Dom G <47725857+DomGarguilo@users.noreply.github.com> Mon Dec 14 13:16:13 2020 -0500
committer GitHub <noreply@github.com> Mon Dec 14 13:16:13 2020 -0500
tree 7bc36efc698c58ffedc6a0ec8cecdb4af5f60223
parent 08748224c77e9e06f6312a24260f375b8ff66139

Fixes #262 - 'uno status' shows shell as "secret(pid)" instead of "shell(pid)" (#263)

* converted to pgrep
* altered script to display correct process name

bin/impl/commands.sh

diff --git a/bin/impl/commands.sh b/bin/impl/commands.sh
index 1a9df24..5ca3e90 100755
--- a/bin/impl/commands.sh
+++ b/bin/impl/commands.sh
@@ -261,11 +261,8 @@
 }
 
 function uno_status_main() {
-  # TODO this should be converted to using pgrep
-  # shellcheck disable=SC2009
-  atmp="$(ps -ef | grep accumulo\\.start | awk '{print $NF "(" $2 ")"}' | tr '\n' ' ')"
-  # shellcheck disable=SC2009
-  htmp="$(ps -ef | grep -e hadoop\\.hdfs -e hadoop\\.yarn | tr '.' ' ' | awk '{print $NF "(" $2 ")"}' | tr '\n' ' ')"
+  atmp="$(pgrep -f accumulo\\.start -a | awk '{pid = $1;for(i=1;i<=NF;i++)if($i=="org.apache.accumulo.start.Main")print $(i+1) "("pid")"}' | tr '\n' ' ')"
+  htmp="$(pgrep -f hadoop\\. -a | tr '.' ' ' | awk '{print $NF "(" $1 ")"}' | tr '\n' ' ')"
   ztmp="$(pgrep -f QuorumPeerMain | awk '{print "zoo(" $1 ")"}' | tr '\n' ' ')"
 
   if [[ -n $atmp || -n $ztmp || -n $htmp ]]; then
@@ -273,7 +270,7 @@
     [[ -n $ztmp ]] && echo "ZooKeeper processes running: $ztmp"
     [[ -n $htmp ]] && echo "Hadoop processes running: $htmp"
   else
-    echo "No components runnning."
+    echo "No components running."
   fi
 }