Skip to content

Fix ContinuationSequenceNumber in SubscribeToShard #12981

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

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

Conversation

NigelRook
Copy link

Motivation

Currently, when using SubscribeToShard, SubscribeToShardEvents will always have a ContinuationSequenceNumber of 0.

#10000

Changes

When using Kinesis SubscribeToShard, ContinuationSequenceNumber will now match the sequence number of the last record sent, rather than 0

Testing

Tested locally with a WIP consumer

@localstack-bot
Copy link
Contributor

localstack-bot commented Aug 8, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link
Contributor

@localstack-bot localstack-bot left a comment

Choose a reason for hiding this comment

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

Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.

@NigelRook
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@NigelRook
Copy link
Author

recheck

@NigelRook NigelRook force-pushed the fix-continuation-sequence-numbner branch from 950217c to 82651e8 Compare August 8, 2025 16:40
localstack-bot added a commit that referenced this pull request Aug 8, 2025
@gregfurman gregfurman self-requested a review August 18, 2025 09:07
@gregfurman gregfurman added aws:kinesis Amazon Kinesis semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases labels Aug 18, 2025
@gregfurman gregfurman added this to the 4.8 milestone Aug 18, 2025
@gregfurman gregfurman force-pushed the fix-continuation-sequence-numbner branch from 82651e8 to c7ea85c Compare August 18, 2025 09:09
@gregfurman gregfurman force-pushed the fix-continuation-sequence-numbner branch from c7ea85c to a8a64a4 Compare August 18, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws:kinesis Amazon Kinesis semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Kinesis Subscriber returns SubscribeToShardEvent with invalid sequence number of 0
3 participants