Skip to content

keyName - if this is not ObjectId it silently fails - #88

Merged
Dominik-Matysiak-gravity merged 2 commits into
mainfrom
develop
Jul 14, 2026
Merged

keyName - if this is not ObjectId it silently fails#88
Dominik-Matysiak-gravity merged 2 commits into
mainfrom
develop

Conversation

@Dominik-Matysiak-gravity

Copy link
Copy Markdown
Contributor

Verified that partitioning works with any key type not only _id
Fixed DELETE events with custom partition key by adding fullDocumentBeforeChangeKey
Added tests
Updated README

Dominik-Matysiak-gravity and others added 2 commits July 14, 2026 14:10
…ame-different-than-objectId

Verified that partitioning works with any key type not only _id
Fixed DELETE events with custom partition key by adding fullDocumentBeforeChangeKey
Added tests
Updated README

@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.

This PR successfully extends the library to support custom partition keys beyond ObjectId types. The implementation correctly addresses DELETE event handling by adding fullDocumentBeforeChange support to the partition matching logic. The comprehensive test suite validates String/Integer keys, partition consistency, and edge cases. Documentation clearly explains the new requirements for DELETE events with custom keys. No blocking issues found.


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 b3ea9f0 into main Jul 14, 2026
5 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