Ana-Maria Satirbasa

Results 12 comments of Ana-Maria Satirbasa

Hello @debora-ito, It is important to note that the `@DynamoDbAutoGeneratedUuid` annotation is not in scope to be used with partition keys. According to the documentation, this annotation is intended only...

Hello @L-Applin, The issue should be fixed now. We are waiting for the workflows to be approved from a maintainer. Can you please review the Pull Request again and let...

Hello @marcusvoltolim, Thank you very much for your review. I have added tests for `@DynamoDbSortKey` as you suggested. The test class (AutoGeneratedKeyRecordTest) now covers all 4 supported key types with...

> Please refer to my comments on the PR. Hello, @RanVaknin! Thank you very much for your feedback! I am in progress with the changes.

Hello @RanVaknin, Thank you very much for the feedback! I've addressed all three points you raised: ### 1. Documentation Updates for UpdateBehavior.WRITE_IF_NOT_EXISTS I've updated the documentation in both `@DynamoDbAutoGeneratedKey` annotation...

Hello @RanVaknin, Thank you for your review! I have updated the Javadoc and added tests for TransactWriteItems and BatchWriteItems in AutoGeneratedKeyRecordTest.java.

> This is a breaking change right ? > > What if currently user was using this API to delete any API m and such cases we will start getting...

Hello @joviegas,   From this standpoint indeed it would be a breaking change. To address the potential breaking change and ensure full backward compatibility while introducing optimistic locking support, we propose...

Hello, @joviegas We understand your concern and we've analyzed your proposal (defining a feature flag for optimistic locking on delete operations at extension level - VersionRecordExtension.java)   We have identified the...

Hello @joviegas, Thanks for your feedback and the request for consistency across all operations. Just to clarify, we have provided two alternatives for how the optimistic locking flag could be...