Skip to content

[Hotfix] Block followership forever for NoopHighAvailabilityContainer#4052

Merged
zhoujinsong merged 3 commits intoapache:masterfrom
zhoujinsong:fix-noop-highavailable
Jan 20, 2026
Merged

[Hotfix] Block followership forever for NoopHighAvailabilityContainer#4052
zhoujinsong merged 3 commits intoapache:masterfrom
zhoujinsong:fix-noop-highavailable

Conversation

@zhoujinsong
Copy link
Contributor

@zhoujinsong zhoujinsong commented Jan 20, 2026

Why are the changes needed?

This PR tried to fix a startup issue when high available is not enabled.

Brief change log

  • As titled.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions bot added the module:ams-server Ams server module label Jan 20, 2026
Copy link
Contributor

@majin1102 majin1102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhoujinsong zhoujinsong merged commit c69eff4 into apache:master Jan 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-server Ams server module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants