adoroszlai commented on a change in pull request #2356:
URL: https://github.com/apache/ozone/pull/2356#discussion_r665142695
##########
File path:
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/shell/TestOzoneShellHA.java
##########
@@ -139,7 +139,6 @@ public static void init() throws Exception {
.setOMServiceId(omServiceId)
.setNumOfOzoneManagers(numOfOMs)
.build();
- conf.setQuietMode(false);
Review comment:
@smengcl I don't understand the importance of keeping quiet mode
disabled. The config file is not really required as everything works in the
default quiet mode. What is the benefit of adding the empty config file?
I have merged the change to unblock follow-up work on protobuf upgrade. If
you have any concerns about the change to this single test, please file a new
issue and we can discuss/address it there.
--
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]
