You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ArbaazKhan1 (via GitHub)" <gi...@apache.org> on 2025/08/21 16:38:10 UTC
[PR] Deprecated the usage of hostnames in StopCommand [accumulo]
ArbaazKhan1 opened a new pull request, #5812:
URL: https://github.com/apache/accumulo/pull/5812
closes [issue #5761 ](https://github.com/apache/accumulo/issues/5761)
I Deprecated `stopTabletServer` since it will read a tserver property from config and use that for the port, which leads to undesirable side affects. I've added a warning in the usage of hostnames in `StopServers` and added a note in the description of the Stop command.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Deprecated the usage of hostnames in StopCommand [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner merged PR #5812:
URL: https://github.com/apache/accumulo/pull/5812
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Deprecated the usage of hostnames in StopCommand [accumulo]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #5812:
URL: https://github.com/apache/accumulo/pull/5812#issuecomment-3238851881
@keith-turner You approved this. Is this okay to merge?
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Deprecated the usage of hostnames in StopCommand [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on code in PR #5812:
URL: https://github.com/apache/accumulo/pull/5812#discussion_r2298512151
##########
server/base/src/main/java/org/apache/accumulo/server/util/Admin.java:
##########
@@ -673,6 +673,9 @@ private static void stopServers(final ServerContext context, List<String> server
if (!hostOnly.isEmpty()) {
// The old impl of this command with the old behavior
+ log.warn("Stopping by hostname is no longer supported\n\n"
+ + "please use <host:port> instead.\n"
+ + "Top stop all services on host, run 'accumulo admin serviceStatus' to list all host:port values for that host and pass those to 'accumulo admin stop'");
Review Comment:
```suggestion
+ "To stop all services on host, run 'accumulo admin serviceStatus' to list all host:port values and filter for that host and pass those to 'accumulo admin stop'");
```
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org