DanielLeens commented on issue #10425:
URL: https://github.com/apache/seatunnel/issues/10425#issuecomment-6037720380

   Thanks for picking up the AmazonDynamoDB source slice. I checked current 
`dev` before treating this as available work: the source factory still requires 
one `table` and `schema`, `AmazonDynamoDBSource` produces one `CatalogTable`, 
and the earlier #10497 work is multi-table **sink** support only. I also 
checked open PRs and found your #12665, so there is no separate competing 
source implementation to start.
   
   The PR's direction is the right one for this umbrella: use `tables_configs` 
for independently declared tables/schemas, keep connection options at source 
scope, carry table identity through splits and emitted rows, and preserve 
existing single-table checkpoint restore semantics. The reader's duplicate-page 
fix should stay explicitly tested as a separate behavioral correction. Please 
keep the feature in #12665 rather than opening another implementation PR; its 
Build is successful but it still needs a full compatibility, split/state, 
routing and E2E review before the tracker can mark the source complete.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to