dlmarion commented on code in PR #5094:
URL: https://github.com/apache/accumulo/pull/5094#discussion_r1854290856
##########
assemble/bin/accumulo-cluster:
##########
@@ -573,49 +573,32 @@ function stop_all() {
}
function stop_here() {
- # Determine hostname without errors to user
- hosts_to_check=("$(hostname -a 2>/dev/null | head -1)" "$(hostname -f)")
-
if echo "${TSERVER_HOSTS}" | grep -Eq 'localhost|127[.]0[.]0[.]1'; then
Review Comment:
> I could change the structure of control_service to accept a --local flag
which would never allow the SSH option to run
This makes sense to me. Obviously if the user is executing a command for the
local host, we should not be ssh'ing.
--
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]
