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

   Thanks for the update, @goutamadwant — good to see the Spark 2.4/3.3/3.5 
regression cases passing in d3d34f292.
   
   F1, F2, F5 and F6 are already confirmed resolved against 
2b832dc0e826f4fd456946f525ade7d83ddb4bbd, so nothing more needed there. A few 
remaining points before I sign off:
   
   1. **F3** — how is the reuse of the Spark 3.3 translation layer on Spark 3.5 
handled now (runtime check, dedicated module, or documented limitation)?
   2. **F4** — is only one SLF4J binding shipped in the starter logging jars 
now, or are the two kept apart per launcher classpath?
   3. **F7** — do the new regression cases exercise the micro-batch streaming 
path on Spark 3.5, or only batch? If streaming is still uncovered, either a 
streaming E2E case or a doc caveat would close this.
   4. **F8** — please state which starter jar Spark 3.4.x users should pick, or 
explicitly mark 3.4.x as untested in the docs.
   
   Also, please ping here once the full container E2E finishes on the fresh run 
so we can confirm the S3 image failure is gone after the dev sync.
   
   <!-- streview-comment:1059 -->


-- 
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