errose28 commented on code in PR #7220:
URL: https://github.com/apache/ozone/pull/7220#discussion_r1767101160
##########
hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/OzoneClientConfig.java:
##########
@@ -207,6 +207,14 @@ public enum ChecksumCombineMode {
tags = ConfigTag.CLIENT)
private long excludeNodesExpiryTime = 10 * 60 * 1000;
+ @Config(key = "ec.reconstruct.validation",
+ defaultValue = "false",
+ description = "The flag whether to validate that EC reconstruction tasks
" +
+ "reconstruct target containers correctly. When validation fails, " +
+ "reconstruction tasks will fail.",
+ tags = ConfigTag.CLIENT)
+ private boolean ecReconstructValidation = false;
Review Comment:
Are there any drawbacks to having this feature enabled? If we commit it as
disabled it will likely never be turned on and we may get burned by a
correctness issue this could have caught. If it is relatively safe we may not
even want/need a config key to disable it.
--
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]
