goutamadwant commented on PR #11307: URL: https://github.com/apache/seatunnel/pull/11307#issuecomment-6387103313
@SEZ9 @DanielLeens Updated the PR description. Please re-review this PR. Changes: - Adds a separate Spark 3.5.8 starter and distribution artifacts, preserving the Spark 3.3 path. - Adds runtime row-encoder compatibility, safer Unix argument handling, Windows exit-code handling and packaged logging checks. - Updates connector E2E fixtures and EN/ZH docs, including Spark 3.4 guidance and compatibility limits. - Adds real Spark 3.5.8 batch, bounded micro-batch sink and packaged Unix launcher tests. Local Java 11 validation on `d8045875a0`: 22 Spark 3.5 unit tests passed, packaged tests 6 passed/4 Windows-only skips, and 11 adjacent Spark 3.3 tests passed. Pending: green CI on the merged head, native Windows packaged-launcher execution, and current-head Spark 3.5 connector/deployment validation. Local checks were not rerun after the latest dev merge. The micro-batch test covers the sink adapter—not streaming through the public starter, checkpoint restore or exactly-once guarantees. Please share any remaining concerns. Thanks. -- 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]
