ddanielr opened a new pull request, #5165:
URL: https://github.com/apache/accumulo/pull/5165

   Fixes a bug in accumulo-cluster where the eval statement was treating the 
ssh command as a separate arg string
   instead of a single command string with contents
   
   Fixes an bug in accumulo-service where if a single running service of the 
desired type was encountered, the script would exit
   
   Also removes the dependency on bash 4.4. 
   
   these changes make the ssh command look like the following:
   
   dry-run output: 
   `ssh -qnf -o ConnectTimeout=2 localhost bash -c 'ACCUMULO_CLUSTER_ARG=2 
/workspace/fluo-uno/install/accumulo-4.0.0-SNAPSHOT/bin/accumulo-service 
"tserver" "start" "-o" "general.process.bind.addr=localhost" "-o" 
"tserver.group=default" '`
   
   `set -v` output when running the actual command:
   `ssh -qnf -o ConnectTimeout=2 localhost bash\ -c\ \'ACCUMULO_CLUSTER_ARG=2\ 
/workspace/fluo-uno/install/accumulo-4.0.0-SNAPSHOT/bin/accumulo-service\ 
\"tserver\"\ \"start\"\ \"-o\"\ \"general.process.bind.addr=localhost\"\ 
\"-o\"\ \"tserver.group=default\"\ \'`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to