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.

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

refactor(controller): consider brokers that has recently CONTROLLED_SHUTDOWN as SHUTTING_DOWN #2261

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

Chillax-0v0
Copy link
Contributor

No description provided.

@Chillax-0v0 Chillax-0v0 self-assigned this Jan 8, 2025
@Chillax-0v0 Chillax-0v0 force-pushed the feat-broker-state-shutting-down branch from 61bbe15 to 7a34357 Compare January 8, 2025 09:59
@Chillax-0v0 Chillax-0v0 force-pushed the feat-broker-state-shutting-down branch from 7a34357 to 100a745 Compare January 8, 2025 10:01
@Chillax-0v0 Chillax-0v0 merged commit b52e27d into main Jan 8, 2025
6 checks passed
@Chillax-0v0 Chillax-0v0 deleted the feat-broker-state-shutting-down branch January 8, 2025 10:24
Chillax-0v0 added a commit that referenced this pull request Jan 8, 2025
…SHUTDOWN` as `SHUTTING_DOWN` (#2261)

* refactor(controller): consider brokers that has recently `CONTROLLED_SHUTDOWN` as `SHUTTING_DOWN`

Signed-off-by: Ning Yu <[email protected]>

* test: test `BrokerHeartbeatManager#brokerState`

Signed-off-by: Ning Yu <[email protected]>

* revert(NodeState): revert `SHUTDOWN` and `SHUTTING_DOWN` to `FENCED` and `CONTROLLED_SHUTDOWN`

Signed-off-by: Ning Yu <[email protected]>

---------

Signed-off-by: Ning Yu <[email protected]>
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.

2 participants