Skip to content

feat: add restarted field to supervisor update API response#19349

Open
abhishekrb19 wants to merge 3 commits intoapache:masterfrom
abhishekrb19:update_supervisor_response
Open

feat: add restarted field to supervisor update API response#19349
abhishekrb19 wants to merge 3 commits intoapache:masterfrom
abhishekrb19:update_supervisor_response

Conversation

@abhishekrb19
Copy link
Copy Markdown
Contributor

@abhishekrb19 abhishekrb19 commented Apr 18, 2026

Summary

Adds a restarted boolean field to the supervisor POST endpoint response to indicate whether the supervisor was actually restarted when skipRestartIfUnmodified is set to true.

When using the skipRestartIfUnmodified parameter, callers had no way to determine from the response whether the supervisor was actually restarted or skipped. This should provide feedback to API callers about the supervisor restart status.

Release note

Adds a restarted boolean field to the supervisor POST endpoint response to indicate whether the supervisor was actually restarted.

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • been tested in a test Druid cluster.

Including this in the response will help indicate if the superivsor was restarted or not
if skipRestartIfUnmodified is set to true
@abhishekrb19 abhishekrb19 force-pushed the update_supervisor_response branch from 4f4b81f to 9b527b5 Compare April 18, 2026 00:05
@abhishekrb19 abhishekrb19 force-pushed the update_supervisor_response branch from 491ad48 to fa52a27 Compare April 18, 2026 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant