adoroszlai commented on PR #8157: URL: https://github.com/apache/ozone/pull/8157#issuecomment-2846551538
@jojochuang @peterxcli This caused [build failure](https://github.com/apache/ozone/actions/runs/14790150556/job/41525587482#step:16:24) on `master` due to other recent changes around OM metadata classes. The compile error can be fixed by simply adding the required import, but the test is failing after that. Thus I had to revert this change, sorry. As a sanity check, I recommend running local build and checkstyle/pmd before merging PRs. ``` gh pr checkout 8157 git merge --no-commit origin/master mvn -DskipRecon -DskipShade -DskipTests clean verify hadoop-ozone/dev-support/checks/checkstyle.sh hadoop-ozone/dev-support/checks/pmd.sh ``` (Note: check exit code of pmd, Maven always shows success.) When there are related changes on `master`, I also recommend merging it into the PR branch to trigger complete CI. -- 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]
