goutamadwant commented on PR #11307:
URL: https://github.com/apache/seatunnel/pull/11307#issuecomment-5234156298

   @SEZ9 @DanielLeens I rechecked the current implementation against the four 
points and synced the branch with latest dev
   
   The existing Spark 3.3 starter version property and runtime path remain 
unchanged. Spark 3.5.8 has its own version property starter artifact launch 
scripts and E2E container. The shared translation code is reused only where the 
APIs are compatible and SparkRowEncoder selects the correct API for the Spark 
version on the classpath
   
   The existing Spark 3 E2E path is still present and the Spark 3.5 container 
uses the 3.5.8 image. The root user setup and null safe getDockerCommand 
handling are also included
   
   After the dev sync I reran the focused Spark starter tests on JDK 11. The 
Spark 3.3 and 3.5 starter tests pass with 7 tests
   
   Could you please review the latest head and let me know if any specific part 
of the parallel 3.5 track is still missing. 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]

Reply via email to