Skip to content

Modified Endpoints for multiple managers and new compaction summary#6251

Open
dlmarion wants to merge 1 commit intoapache:mainfrom
dlmarion:6247-compaction-summary
Open

Modified Endpoints for multiple managers and new compaction summary#6251
dlmarion wants to merge 1 commit intoapache:mainfrom
dlmarion:6247-compaction-summary

Conversation

@dlmarion
Copy link
Copy Markdown
Contributor

Closes #6247

@dlmarion dlmarion added this to the 4.0.0 milestone Mar 25, 2026
@dlmarion dlmarion requested a review from DomGarguilo March 25, 2026 21:23
@dlmarion dlmarion self-assigned this Mar 25, 2026
@dlmarion
Copy link
Copy Markdown
Contributor Author

@DomGarguilo - this changes two endpoints. Let me know if you want any changes in the response object formats and I can make them.

@DomGarguilo
Copy link
Copy Markdown
Member

@DomGarguilo - this changes two endpoints. Let me know if you want any changes in the response object formats and I can make them.

I think with these changes the JSON will be something like:

  {
    "type=MANAGER;group=default;address=localhost:9995": [
      { "...": "metric..." },
      { "...": "metric..." }
    ]
  }

That will work but is not quite as nice to parse. I'm thinking it might be better to send a list of objects instead of a map. so something like this:

  [
    {
      "server": {
        "type": "MANAGER",
        "resourceGroup": "default",
        "address": "localhost:9995"
      },
      "metrics": [
        { "...": "metric..." },
        { "...": "metric..." }
      ]
    }
  ]

Not exactly sure what the code will look like but could create something like record ServerIdView(String type, String resourceGroup, String address) for reuse in the other endpoints that need the ServerId too

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.

Add monitor rest endpoint for running compaction stats

2 participants