Skip to content

Develop - #91

Merged
Dominik-Matysiak-gravity merged 2 commits into
mainfrom
develop
Aug 24, 2026
Merged

Develop#91
Dominik-Matysiak-gravity merged 2 commits into
mainfrom
develop

Conversation

@Dominik-Matysiak-gravity

Copy link
Copy Markdown
Contributor

Fixed a bug where change stream events without the partition key matched no partition and were silently dropped. Replaced the three $or branches with a cascading expression that picks the first available key source and falls back to DEFAULT_PARTITION = 0, leaving routing unchanged when the key is present.

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The fix correctly addresses the silent data loss issue by routing documents without partition keys to partition 0 instead of filtering them out. The cascading conditional expression provides a clean fallback mechanism, and the comprehensive test coverage validates the behavior across multiple scenarios including INSERT, UPDATE, and DELETE operations with custom keys.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@Dominik-Matysiak-gravity
Dominik-Matysiak-gravity merged commit 3ca96bd into main Aug 24, 2026
8 of 9 checks passed
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