ajantha-bhat commented on PR #16946: URL: https://github.com/apache/iceberg/pull/16946#issuecomment-5322868691
I was on holiday. I am back now. I gave another shot at it as @kevinjqliu suggested. Please take a look. The planner now only controls the Spark version axis: - for PRs, it narrows to one Spark version only when every changed file is under the same known `spark/v<version>/` path - mixed Spark versions, unknown Spark versions, workflow/script/build changes, non-version Spark paths, missing changed-file detection, and non-PR runs all use the full Spark matrix I also removed the shared helper and dropped the generated Scala/exclude handling. Spark/Scala compatibility and the explicit PR JDK behavior now stay visible in the workflow matrix, so the PR is only about Spark-version selection. I validated the planner cases locally, including same-version Spark changes, mixed Spark versions, unknown paths, workflow changes, non-PR runs, and this PR’s own diff. -- 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]
