Skip to content

Conversation

@dlmarion
Copy link
Contributor

This change removes the StatusSummary.serviceType
field from the json serialization. The serviceType field is redundant in the json output as the same
information is used as the key in a map that contains the StatusSummary object.

This change also removes the StatusSummary.serviceByGroups field from the json serialization using a custom
ExclusionStrategy.

Closes #6088

This change removes the StatusSummary.serviceType
field from the json serialization. The serviceType
field is redundant in the json output as the same
information is used as the key in a map that contains
the StatusSummary object.

This change also removes the StatusSummary.serviceByGroups
field from the json serialization using a custom
ExclusionStrategy.

Closes apache#6088
@dlmarion dlmarion added this to the 4.0.0 milestone Jan 29, 2026
@dlmarion dlmarion self-assigned this Jan 29, 2026
@dlmarion
Copy link
Contributor Author

The output before this change is shown at #6088 (comment)

The output after this change is below:

./accumulo admin serviceStatus --json
...
        "T_SERVER": {
            "resourceGroups": {
                "default": 1
            },
            "serviceCount": 1,
            "errorCount": 0
        }
...
./accumulo admin serviceStatus --json --showHosts
...
        "T_SERVER": {
            "resourceGroups": {
                "default": 1
            },
            "serviceByGroups": {
                "default": [
                    "localhost:9997"
                ]
            },
            "serviceCount": 1,
            "errorCount": 0
        }
...

dlmarion and others added 2 commits January 29, 2026 15:02
@dlmarion dlmarion merged commit 20da5e1 into apache:main Jan 29, 2026
8 checks passed
@dlmarion dlmarion deleted the 6088-fix-service-status-json branch January 29, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

serviceStatus json output includes some redundant information and missing information

2 participants