Skip to content

Remove direct dynamodb_lock interactions#222

Open
Sharayu1418 wants to merge 3 commits intodelta-io:mainfrom
Sharayu1418:remove-dynamodb-lock
Open

Remove direct dynamodb_lock interactions#222
Sharayu1418 wants to merge 3 commits intodelta-io:mainfrom
Sharayu1418:remove-dynamodb-lock

Conversation

@Sharayu1418
Copy link

Removes the direct dynamodb_lock dependency as it's no longer needed
with deltalake >= 0.17.0, which handles S3 DynamoDB locking internally
via S3DynamoDbLogStore.

Changes:

  • Removed dynamodb_lock dependency from Cargo.toml
  • Removed dynamodb_lock imports and related code from src/dead_letters.rs
  • Updated README.adoc to reflect locking is now handled internally

@rtyler rtyler self-requested a review March 2, 2026 14:05
@rtyler rtyler enabled auto-merge (rebase) March 2, 2026 14:05
Signed-off-by: R. Tyler Croy <rtyler@brokenco.de>
@rtyler rtyler force-pushed the remove-dynamodb-lock branch from 32a7fb7 to 7a9cad1 Compare March 8, 2026 17:23
Signed-off-by: R. Tyler Croy <rtyler@brokenco.de>
@rtyler rtyler force-pushed the remove-dynamodb-lock branch from 7a9cad1 to 21ed093 Compare March 9, 2026 20:53
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