Fix Kafka patch filter precondition handling - #50053
Fix Kafka patch filter precondition handling#50053Arnab Nandy (arnabnandy7) wants to merge 1 commit into
Conversation
|
Azure Pipelines: Successfully started running 2 pipeline(s). 33 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
67255a6 to
05db6bc
Compare
There was a problem hiding this comment.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Note
This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.
There was a problem hiding this comment.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Note
This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.
|
Thank you for your contribution Arnab Nandy (@arnabnandy7)! We will review the pull request and get back to you soon. |
Signed-off-by: Arnab Nandy <arnab_nandy7@yahoo.com>
05db6bc to
57463b8
Compare
Description
Fixes #49944.
Updates the Cosmos DB Kafka sink connector so
ItemPatchwrites treat412 Precondition Failedresponses as successful no-op skips when a non-emptyazure.cosmos.sink.write.patch.filterpredicate is configured.The change applies consistently to both bulk and point-write paths. Patch operations without a filter continue to treat
412responses as failures. Inline comments document that this behavior should be revisited if etag/If-Matchsupport is added to patch operations.The existing Kafka sink patch integration test now covers conditional patch replay for both bulk and point writers. The configuration reference and CHANGELOG have also been updated.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines