ArafatKhan2198 commented on PR #4534: URL: https://github.com/apache/ozone/pull/4534#issuecomment-1505384456
> > Therefore, I have decided to incorporate this small name change in this PR. > > It would be better to avoid incorporating unrelated changes. > > 1. We already had successful CI for this PR, could have been merged. Now we need to trigger another run. > 2. Commit message would be confusing (fix flaky test vs. rename JSON element). > 3. CI needs to run fewer checks if only unit test is changed. Understood, I have removed those changes. Thanks for the feedback -- 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]
