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

Includes conditions expressions for transactions Put. #1776

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

preissler
Copy link

Hello Scanamo maintainers,

I've made some enhancements to the Scanamo Scala library to improve its functionality. Below is a detailed summary of the changes:

Include conditions for put items in ScanamoTransactWriteRequest:
Implemented the ability to specify request conditions for put items within ScanamoTransactWriteRequest. This enhancement empowers users to define conditions that must be met for transactional put operations to succeed, it complements the already implemented support for transactions conditions .

Exposes TransactionCanceledException:
This enhancement strengthens error handling capabilities, enabling users to distinguish between different transaction cancellation scenarios, thereby facilitating improved troubleshooting and error resolution.

These changes have been running successfully in our production environment since mid-March without any observed issues. This feature is particularly crucial for us as it enables the implementation of transactions and optimistic locks with Scanamo.

Thank you.

…ransactionCanceledException for client transactWriteItems calls.
…ransactionCanceledException for client transactWriteItems calls.
@preissler preissler requested a review from rtyley as a code owner May 6, 2024 16:28
@asarturas
Copy link

Hi @rtyley, I wonder what are your thoughts and/or if you had a chance to consider this contribution? Would be great to have fuller transaction support in main Scanamo package.

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.

None yet

2 participants