hutiefang76 commented on PR #12938: URL: https://github.com/apache/seatunnel/pull/12938#issuecomment-6335907855
Thanks for the invitation, @JeremyXin. Let's keep the scope and merge-order decisions here so the reviewers can follow them. I'd keep the JDBC fix in #12875 and #12938 focused on the archived LLM fixture. We can still validate the combined changes before merging either PR. My suggested order is to merge #12875 first, then update #12938 and remove the overlapping JDBC commit. #12875 also includes the factory-level check that the option remains accepted by the source and rejected by the sink. That keeps the JDBC fix and its regression coverage together, with one review history for each change. <sub>Drafted by an AI assistant for @hutiefang76.</sub> -- 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]
