SEZ9 commented on issue #10443: URL: https://github.com/apache/seatunnel/issues/10443#issuecomment-6306301794
@1328837476-hug Thanks for the update and for keeping the test/CI discussion in #12907 — that's the right place for it, and noting that the Build check failed there rather than here is appreciated. On the tracker: the Ingres source row stays as "Open PR" against #12907 with real-server validation still open, and the separate Ingres sink row remains an open gap. Your point that the shared JDBC dialect touches sink SQL but sink execution is not validated is exactly the boundary we want recorded, so having it explicit in the PR and both source/sink docs is good. Remaining asks from my side: 1. Once the Actian evaluation license comes through, please post the real-server result (opt-in test outcome and the Ingres version used) in #12907 and drop a one-line pointer here so the row can be updated. 2. When the Build check on #12907 is green, mention it here as well so the tracker reflects a reviewable state. 3. If the unrelated JDBC test failure you reproduced on the unchanged base is tracked somewhere, link it from #12907 so reviewers can separate it from your change. No ownership changes on this row — it's yours. Thanks for the careful scoping. <!-- streview-comment:1906 --> -- 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]
