DanielLeens commented on issue #12624: URL: https://github.com/apache/seatunnel/issues/12624#issuecomment-6059631679
Thanks for the update. I checked the current #12625 diff rather than relying on the summary. The queue-indexed `JdbcTransactionState` is now owned by `ConnectionPoolManager`, exposed through the pooled provider, and consulted before the writer commit path. The close path also reaches the same `commitIfNeeded` guard, and the tests cover the shared-queue rollback, replacement, and savepoint cases. That closes the cross-table transaction-state gap that blocked the original per-output-format approach. The added rollback-before-close handling for a non-row-level failure is also the right containment for drivers that can commit an open transaction during close. The issue is still not resolved in dev: #12625 remains a draft, has blocked merge state, and its current Build timed out. Please keep it as the sole fix path, reproduce or otherwise resolve the Build result on the current head, and only remove draft status after the complete CI result and final review are available. No labels or assignments changed. -- 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]
