laserninja opened a new pull request, #13161: URL: https://github.com/apache/gravitino/pull/13161
### What changes were proposed in this pull request? Reject different job artifact sources with the same staging filename before fetching any files. Check the executable and all applicable scripts, jars, files, and archives together, while allowing repeated references to the same source. ### Why are the changes needed? A script with the same filename as the executable currently overwrites it and causes the job to run the wrong script. Fixes #13158 ### Does this PR introduce _any_ user-facing change? Conflicting artifact filenames now fail with an explanatory error before staging or launching a job. No public API or configuration changes. ### How was this patch tested? `JAVA_HOME=<JDK17> ./gradlew :core:spotlessApply :core:test --tests 'org.apache.gravitino.job.*' :core:build -PskipITs --max-workers=2` passed (92 tests). New regression tests failed before the fix. The original local-process reproduction ran SECOND instead of FIRST; after the fix, runtime preparation rejects the collision with an empty staging directory. Full server integration tests were not run. -- 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]
