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

Offset fetch request does not support Kafka-committed offsets #421

Closed
toddpalino opened this issue Jul 2, 2015 · 3 comments
Closed

Offset fetch request does not support Kafka-committed offsets #421

toddpalino opened this issue Jul 2, 2015 · 3 comments

Comments

@toddpalino
Copy link
Contributor

The offset fetch request as implemented only supports offsets stored in Zookeeper. I've added support internally for fetching Kafka-committed offsets by adding the basic support needed for the consumer coordinator and adding a separate client method for fetching offsets from Kafka. The separate method is needed because fetching offsets from Kafka vs. from Zookeeper uses different versions of OffsetFetchRequest, even though the wire format is the same.

I've created PR #420 to add this functionality. We should be able to build the rest of the support for the consumer coordinator, including committing offsets to Kafka storage, from there.

@wushujames
Copy link

I'm interested in this functionality as well.

@zorkian
Copy link

zorkian commented Oct 30, 2015

+1. It'd be great to get PR #420 reviewed and merged if possible.

@dpkp
Copy link
Owner

dpkp commented Dec 2, 2015

merged

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

No branches or pull requests

4 participants