Ana-Maria Satirbasa
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...