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-14462; [16/N] Add CoordinatorLoader and CoordinatorPlayback interfaces #13794

Merged
merged 3 commits into from Jun 2, 2023

Conversation

dajac
Copy link
Contributor

@dajac dajac commented Jun 1, 2023

This patch adds the CoordinatorLoader and the CoordinatorPlayback interfaces. The former is used to load (or rebuild) the coordinator state by replaying all the records stored in the partition. The later is the interface that must be implemented by coordinator to support replaying records.

Committer Checklist (excluded from commit message)

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

@jolshan
Copy link
Contributor

jolshan commented Jun 1, 2023

Let's add a brief description to explain the rationale for these interfaces to the description

/**
* The Replayable interface.
*/
public interface Replayable<U> {
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to name it more specifically to the coordinators?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have renamed it to CoordinatorPlayback. Does it sound good to you?

@dajac dajac changed the title KAFKA-14462; [16/N] Add CoordinatorLoader and Replayable interfaces KAFKA-14462; [16/N] Add CoordinatorLoader and CoordinatorPlayback interfaces Jun 2, 2023
@dajac dajac requested a review from jolshan June 2, 2023 09:32
Copy link
Contributor

@jolshan jolshan left a comment

Choose a reason for hiding this comment

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

Thanks!

@dajac dajac merged commit e5d67b8 into apache:trunk Jun 2, 2023
1 check failed
@dajac dajac deleted the KAFKA-14462-16 branch June 2, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants