dmefs opened a new pull request, #4120:
URL: https://github.com/apache/datafusion-comet/pull/4120

   ## Which issue does this PR close?
   
   Closes #4101 .
   
   ## Rationale for this change
   - Iceberg's own CI does not exhaustively cross JDK with every Spark version; 
we are overcovering relative to upstream.
   - JDK 11 vs 17 differences that affect Comet are caught elsewhere: 
pr_build_linux.yml and spark_sql_test.yml both run JDK 11 + 17 against multiple 
Spark versions.
   - The Iceberg suites are about Iceberg/Comet integration, not JVM-level 
differences; the same JDK delta is unlikely to surface there but not in the 
broader matrix.
   - Frees CI capacity to add Spark 4.0.1 and 4.1.1 to the matrix without a net 
increase in PR runtime.
   
   ## What changes are included in this PR?
   The java-version: [11, 17] axis is removed from the base matrix of all three 
test jobs (iceberg-spark, iceberg-spark-extensions, iceberg-spark-runtime) and 
replaced with an `include:` block that injects the appropriate JDK per Spark 
version:
   
   - Spark 3.4.3 → JDK 11
   - Spark 3.5.8 → JDK 17
   
   This reduces test jobs from 36 to 18 per PR with no changes to step logic or 
variable references.
   
   ## How are these changes tested?
   CI on this PR shows 18 jobs instead of 36.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to