Skip to content

KAFKA-17302: APIs for Follower Fetch from Last Tiered Offset#21153

Merged
kamalcph merged 3 commits intoapache:trunkfrom
abhijeetk88:KAFKA-17302_leaderendpoint
Dec 24, 2025
Merged

KAFKA-17302: APIs for Follower Fetch from Last Tiered Offset#21153
kamalcph merged 3 commits intoapache:trunkfrom
abhijeetk88:KAFKA-17302_leaderendpoint

Conversation

@abhijeetk88
Copy link
Copy Markdown
Contributor

@abhijeetk88 abhijeetk88 commented Dec 15, 2025

In this PR, we are adding the methods required for supporting follower
fetch from Last Tiered Offset.

  • Added new method shouldFetchFromLastTieredOffset in
    AbstractFetcherThread to determine if the folllower should fetch from
    the last tiered offset for the remote storage enabled topic
  • Added new API fetchEarliestPendingUploadOffset to LeaderEndPoint for
    retrieving the offset from where it should start replicating data from
    the leader.

Reviewers: Kamal Chandraprakash kamal.chandraprakash@gmail.com

@abhijeetk88 abhijeetk88 marked this pull request as draft December 15, 2025 13:23
@github-actions github-actions Bot added triage PRs from the community core Kafka Broker storage Pull requests that target the storage module labels Dec 15, 2025
@abhijeetk88 abhijeetk88 changed the title FetchEarliestPendingUploadOffset API in LeaderEndPoint APIs for Follower Fetch from Last Tiered Offset Dec 15, 2025
Comment thread core/src/main/scala/kafka/server/LocalLeaderEndPoint.scala Outdated
Comment thread core/src/test/scala/kafka/server/LocalLeaderEndPointRemoteTest.scala Outdated
@github-actions github-actions Bot removed the triage PRs from the community label Dec 17, 2025
@abhijeetk88 abhijeetk88 force-pushed the KAFKA-17302_leaderendpoint branch from fa3bc8f to f0fa369 Compare December 24, 2025 09:55
@abhijeetk88 abhijeetk88 marked this pull request as ready for review December 24, 2025 10:02
@abhijeetk88 abhijeetk88 changed the title APIs for Follower Fetch from Last Tiered Offset KAFKA-17302: APIs for Follower Fetch from Last Tiered Offset Dec 24, 2025
Copy link
Copy Markdown
Contributor

@kamalcph kamalcph 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 PR!

@kamalcph kamalcph merged commit 0d9ae49 into apache:trunk Dec 24, 2025
35 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Kafka Broker storage Pull requests that target the storage module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants