Skip to content

Fix: Update OffsetCommit CommittedLeaderEpoch to v6 - #1335

Merged
petedannemann merged 3 commits into
segmentio:mainfrom
LeonHartley:main
May 15, 2025
Merged

Fix: Update OffsetCommit CommittedLeaderEpoch to v6#1335
petedannemann merged 3 commits into
segmentio:mainfrom
LeonHartley:main

Conversation

@LeonHartley

Copy link
Copy Markdown
Contributor

No description provided.

@LeonHartley LeonHartley changed the title Update OffsetCommit ComittedLeaderPartition to v6 (#1334) Update OffsetCommit CommittedLeaderEpoch to v6 (#1334) Sep 24, 2024
@LeonHartley LeonHartley changed the title Update OffsetCommit CommittedLeaderEpoch to v6 (#1334) Fix: Update OffsetCommit CommittedLeaderEpoch to v6 Sep 24, 2024
petedannemann
petedannemann previously approved these changes May 14, 2025
@petedannemann
petedannemann self-requested a review May 14, 2025 19:50
@petedannemann

Copy link
Copy Markdown
Contributor

@LeonHartley this looks good but can you merge main into your PR? I see the tests were failing before but those logs have expired in CircleCI. It seems unlikely they are failing now but I just want to verify

@LeonHartley

Copy link
Copy Markdown
Contributor Author

@petedannemann cheers, I have updated with latest main now

@LeonHartley

Copy link
Copy Markdown
Contributor Author

I noticed the TestOffsetCommitRequest tests were now failing due to them using the wrong protocol version too, I've updated those tests now and verified that they pass.

@LeonHartley
LeonHartley force-pushed the main branch 2 times, most recently from 87380c6 to b649bde Compare May 15, 2025 20:09

@petedannemann petedannemann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

@petedannemann
petedannemann merged commit af1725f into segmentio:main May 15, 2025
ashishkf pushed a commit to kloudfuse/kafka-go that referenced this pull request Aug 4, 2026
* Update OffsetCommit CommittedLeaderEpoch to v6, fixes segmentio#1334

* update OffsetCommitRequest test to use correct protocol version
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