DanielLeens commented on issue #13131: URL: https://github.com/apache/seatunnel/issues/13131#issuecomment-6657735712
Thanks for tracing the ownership boundary. Current `dev` (`3cb9efc2a15223372bc1f87a92d027845a8ca800`) still has `AbstractMultiCatalogTransform` own the per-table `transformMap` without an `open()` or `close()` override, while the interface defaults are no-ops. This is therefore a valid shared Transform-V2 lifecycle-correctness issue. A focused PR is welcome. Please keep it independent from the engine-side failure propagation in #13132, while making the wrapper contract explicit: 1. Each owned child receives lifecycle callbacks exactly once in normal startup and shutdown. 2. A child-open failure stops further child initialization and leaves partial initialization cleanup with a deterministic, testable owner. 3. Closing continues across children when appropriate, without hiding the first failure or duplicating callbacks from wrappers that already manage their own children. 4. Map and FlatMap paths, identity/pass-through children, composed rules, and existing serialization behavior remain compatible. Please extend the existing multi-table transform tests rather than adding a parallel test scaffold, and include the successful control case as well as partial-failure and duplicate-callback guards. -- 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]
