Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The admin serviceStatus command needs additional changes from 2.1 #4603

Closed
EdColeman opened this issue May 24, 2024 · 0 comments · Fixed by #4605
Closed

The admin serviceStatus command needs additional changes from 2.1 #4603

EdColeman opened this issue May 24, 2024 · 0 comments · Fixed by #4605
Assignees
@EdColeman
Labels
bug This issue has been verified to be a bug.
Projects
3.1.0

Comments

@EdColeman
Copy link
Contributor

The merge of 2.1 with the service status command is in 8e5d967 is complete because the lock data changed with the use of ServiceLock. The changes are large enough that a separate PR will be used to fix the command.

Currently the command will not print the correct host:port and the counts may be off because it is treating the entire service descriptor as the host:port.
@EdColeman EdColeman added the bug This issue has been verified to be a bug. label May 24, 2024
@EdColeman EdColeman self-assigned this May 24, 2024
@EdColeman EdColeman added this to To do in 3.1.0 via automation May 24, 2024
@EdColeman EdColeman mentioned this issue May 25, 2024
Merged
3.1.0 automation moved this from To do to Done May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EdColeman EdColeman

Labels
bug This issue has been verified to be a bug.
Projects
3.1.0
Done
Accumulo Elasticity
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Admin service status server lock changes EdColeman/accumulo
1 participant
@EdColeman