sodonnel commented on a change in pull request #3217:
URL: https://github.com/apache/ozone/pull/3217#discussion_r832151387
##########
File path:
hadoop-ozone/fault-injection-test/mini-chaos-tests/src/test/java/org/apache/hadoop/ozone/TestMiniChaosOzoneCluster.java
##########
@@ -91,6 +98,16 @@
"--failureInterval will be removed in later versions.")
private static int failureInterval = 300; // 5 minute period between
failures.
+ @Option(names = {"-r", "--replication-type"},
Review comment:
Across various commands we have been adding two options to facilitate EC:
* `-t --type` - for RATIS or EC
* `-r --replication` - the replication that goes with the type - eg ONE,
THREE, rs-3-2-1024k etc.
It would be best to try to keep that convention here and add both options.
Then remove the hardcoded `rs-3-2` added later in the class, as it would allow
it to be more flexible in the future.
--
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]
