SEZ9 commented on issue #12979: URL: https://github.com/apache/seatunnel/issues/12979#issuecomment-6392589793
Thanks for the thorough write-up and measurements. Marking `extraTablePaths` transient so the implicit UID returns to `8769738606872706499`, and rebuilding it in `readObject` (or lazily before `map()` uses it), sounds like a reasonable direction, especially since `setInputCatalogTables` also fails on a deserialized instance and so cannot serve as a repair path. Which UID window to support (pre-#12804 streams vs. streams written between #12804 and the fix) is worth stating explicitly in the PR so the choice is deliberate. Since you already have the fix and stream fixtures ready, I would not hold the handoff open for long: if the author of #12938 does not reply within a few days, please go ahead and open the PR. Please include a serialization round-trip test (deserialize a `ChainedMapTransform`, then exercise both `map()` and `setInputCatalogTables`) and link the PR to this issue, #12804 and #12938. <!-- streview-comment:2014 --> -- 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]
