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

KAFKA-12607: Test case for resigned state vote granting #10510

Merged
merged 2 commits into from
Apr 10, 2021

Conversation

dengziming
Copy link
Member

More detailed description of your change
As discussed in the Jira, ResignedState will transition to VotedState and grant vote, just add some unit tests to verify this transition.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Copy link
Contributor

@hachikuji hachikuji left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the test cases!

@hachikuji hachikuji merged commit db688b1 into apache:trunk Apr 10, 2021
jthurne added a commit to jthurne/kafka that referenced this pull request Apr 12, 2021
* origin/trunk: (27 commits)
  MINOR: Remove some unnecessary cyclomatic complexity suppressions (apache#10488)
  MINOR: Remove gradleSetup from Jenkinsfile (apache#10522)
  MINOR: Use `testRuntimeOnly` instead of `testRuntime` in storage modules (apache#10524)
  KAFKA-12607; Test case for resigned state vote granting (apache#10510)
  HOTFIX: delete removed WindowedStore.put() method (apache#10517)
  KAFKA-8410: Migrate KStream Stateless operators to new Processor API (apache#10381)
  KAFKA-9831: increase max.poll.interval.ms to avoid unexpected rebalance (apache#10301)
  MINOR: Fix typo in docs (apache#10423)
  KAFKA-12449: Remove deprecated WindowStore#put (apache#10293)
  MINOR: Fix left/outer join descriptions in Streams DSL reference topic (apache#10453)
  MINOR: add missing TimeoutException to Producer.send JavaDocs (apache#10495)
  KAFKA-12591; Remove deprecated `quota.producer.default` and `quota.consumer.default` configurations (apache#10427)
  KAFKA-12492: Fix the formatting of example RocksDBConfigSetter (apache#10486)
  KAFKA-12406 Integrate client quotas with KRaft broker (apache#10254)
  KAFKA-12619; Raft leader should expose hw only after committing LeaderChange (apache#10481)
  KAFKA-12457; Add sentinel ID to metadata topic (apache#10492)
  HOTFIX: add missing upgrade docs
  KAFKA-12630: Remove deprecated KafkaClientSupplier#getAdminClient in Streams (apache#10502)
  MINOR: Remove redundant code from ReplicaManagerTest#testFetchMessagesWhenNotFollowerForOnePartition (apache#10501)
  MINOR: Update Crc32Test#testUpdate method with correct Crc32 (apache#10406)
  ...
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