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

Add fetching messages from JetStream #87

Merged
merged 6 commits into from
Oct 31, 2024
Merged

Add fetching messages from JetStream #87

merged 6 commits into from
Oct 31, 2024

Conversation

piotrpio
Copy link
Collaborator

@piotrpio piotrpio commented Oct 8, 2024

This PR adds Consumer.fetch() method for fetching batches of messages. It also adds JetStreamMessage and methods for acking messages and checking metadata.

Signed-off-by: Piotr Piotrowski [email protected]

@piotrpio piotrpio requested a review from Jarema October 8, 2024 15:09
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Show resolved Hide resolved
Sources/JetStream/Consumer+Pull.swift Outdated Show resolved Hide resolved
Sources/JetStream/JetStreamMessage.swift Outdated Show resolved Hide resolved
piotrpio and others added 5 commits October 29, 2024 16:14
Co-authored-by: Tomasz Pietrek <[email protected]>
Co-authored-by: Tomasz Pietrek <[email protected]>
Co-authored-by: Tomasz Pietrek <[email protected]>
Co-authored-by: Tomasz Pietrek <[email protected]>
Signed-off-by: Piotr Piotrowski <[email protected]>
Copy link
Member

@Jarema Jarema left a comment

Choose a reason for hiding this comment

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

LGTM!

@Jarema Jarema merged commit 436ca4a into main Oct 31, 2024
5 checks passed
@Jarema Jarema deleted the consumer-fetch branch October 31, 2024 07:09
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