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.

Sign up for GitHub

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

Jump to bottom

Support seek a MessageId from pulsar module #233

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Support seek a MessageId from pulsar module #233

wants to merge 1 commit into from

Conversation

BewareMyPower
Copy link
Contributor

Currently the seek method of Consumer or Reader can only accept the MessageId from the _pulsar module, which is the underlying C extension that should not be exposed. If a MessageId instance from the pulsar module is passed as the argument, this method will fail.

@BewareMyPower BewareMyPower added the bug Something isn't working label Dec 9, 2024
@BewareMyPower BewareMyPower added this to the 3.6.0 milestone Dec 9, 2024
@BewareMyPower BewareMyPower self-assigned this Dec 9, 2024
@BewareMyPower BewareMyPower marked this pull request as draft December 11, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@merlimat merlimat Awaiting requested review from merlimat

@RobertIndie RobertIndie Awaiting requested review from RobertIndie

@shibd shibd Awaiting requested review from shibd

At least 1 approving review is required to merge this pull request.

Assignees

@BewareMyPower BewareMyPower

Labels
bug Something isn't working
Projects
None yet
Milestone
3.6.0
Development

Successfully merging this pull request may close these issues.
1 participant
@BewareMyPower