davidzollo commented on PR #11479: URL: https://github.com/apache/seatunnel/pull/11479#issuecomment-6394149047
Addressed the current review in `2e7bd766886ebccb4e04c44e9d49092922a78a04`. - CDC transaction buffering preserves the complete record key; short and composite common handles no longer pass through integer-only identity decoding. - Explicit clustered-key metadata separates common handles from non-clustered row IDs. Snapshot and streaming decoding restore only missing PK columns, preserve row-value originals, validate record-key shape, and handle key-only DELETE without changing the existing UPDATE_AFTER contract. - Added short-key/composite/collation/row-kind/metadata/transaction-map regressions and extended the existing real TiDB E2E with nonempty snapshot and transactional row/value reconciliation. EN/ZH documentation now describes the behavior. Scoped Spotless formatting passed for connector and E2E modules, and the complete patch passed independent source review. At remote readback, the exact head and 16-file PR scope matched and the PR was mergeable; the required Build was queued. Functional tests were not run locally under the repository policy, and current-head GitHub CI remains required. -- 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]
