GeorgeJahad commented on pull request #2974: URL: https://github.com/apache/ozone/pull/2974#issuecomment-1021467548
This comment confused me for a while, because it emphasizes the ozone-runner image: *ozone-runner image version is defined in hadoop-ozone/dist/pom.xml. Changing it triggers a complete CI run, although it does not affect unit or integration tests in any way. Actually, hadoop-ozone/dist does not have any Java code, as it assembles other modules after build. So we can skip all Java-specific checks.* I would have left the ozone-runner out of it. To me it is clearer to just say: *hadoop-ozone/dist does not have any Java code, as it assembles other modules after build. So we can skip all Java-specific checks, for any changes in that directory.* -- 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]
