Skip to content

Tests added for follower fetch protocol (part-1)#39

Merged
satishd merged 4 commits into
satishd:tier-follower-fetchfrom
kamalcph:u11390
Sep 20, 2022
Merged

Tests added for follower fetch protocol (part-1)#39
satishd merged 4 commits into
satishd:tier-follower-fetchfrom
kamalcph:u11390

Conversation

@kamalcph

@kamalcph kamalcph commented Sep 20, 2022

Copy link
Copy Markdown
Collaborator

Tests added/updated for the below classes:

  1. LogManagerTest
  2. ProducerStateManagerTest
  3. UnifiedLogTest
  4. RemoteIndexCacheTest
  5. LeaderEpochFileCacheTest

Opensource PR: apache#11390

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.

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)

@kamalcph kamalcph requested a review from satishd September 20, 2022 06:41
Comment thread core/src/main/scala/kafka/log/LogManager.scala

@satishd satishd left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks @kamalcph for the PR, LGTM.

@satishd satishd merged this pull request into satishd:tier-follower-fetch Sep 20, 2022
satishd pushed a commit that referenced this pull request Sep 20, 2022
* Tests added for UnifiedLog, RemoteIndexCache, LeaderEpochFileCache and ProducerStateManager
* RemoteLogManager instantiation test on KafkaServer and BrokerServer added.
* LeaderEpochFileCache assign-without-flush test case added.
* Covered test for UnifiedLog#legacyFetchOffsetsBefore
satishd pushed a commit that referenced this pull request Nov 24, 2022
* Tests added for UnifiedLog, RemoteIndexCache, LeaderEpochFileCache and ProducerStateManager
* RemoteLogManager instantiation test on KafkaServer and BrokerServer added.
* LeaderEpochFileCache assign-without-flush test case added.
* Covered test for UnifiedLog#legacyFetchOffsetsBefore
satishd pushed a commit that referenced this pull request Dec 8, 2022
* Tests added for UnifiedLog, RemoteIndexCache, LeaderEpochFileCache and ProducerStateManager
* RemoteLogManager instantiation test on KafkaServer and BrokerServer added.
* LeaderEpochFileCache assign-without-flush test case added.
* Covered test for UnifiedLog#legacyFetchOffsetsBefore
satishd pushed a commit that referenced this pull request Dec 16, 2022
* Tests added for UnifiedLog, RemoteIndexCache, LeaderEpochFileCache and ProducerStateManager
* RemoteLogManager instantiation test on KafkaServer and BrokerServer added.
* LeaderEpochFileCache assign-without-flush test case added.
* Covered test for UnifiedLog#legacyFetchOffsetsBefore
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