JeremyXin commented on PR #12938: URL: https://github.com/apache/seatunnel/pull/12938#issuecomment-6335749206
@DanielLeens Hi Daniel, thanks for the detailed review. I’ve narrowed the PR title and description to the failure demonstrated by the archived LLM wrapper fixture. The PR no longer claims to restore compatibility for all legacy multi-catalog transforms. I agree that other concrete wrappers may have independent UID mismatches, and that archived chained rules need separate UID and field-initialization work. Those paths have not been validated or fixed here; I’ll track them separately. The JDBC test commit remains in this branch for now. I’ve called out its overlap with \#12875 in the PR description. -- 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]
