sodonnel commented on a change in pull request #2973:
URL: https://github.com/apache/ozone/pull/2973#discussion_r784125914
##########
File path:
hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/OzoneClientConfig.java
##########
@@ -131,6 +131,14 @@
tags = ConfigTag.CLIENT)
private int maxECStripeWriteRetries = 10;
+ @Config(key = "exclude.nodes.expiry.time",
+ defaultValue = "600000",
+ description = "Ozone EC client to remove the node from the exclude" +
Review comment:
Should the expiry apply to both EC and non-EC clients?
The description doesn't read nicely for me - can we reword to something like:
> Time after which an excluded node is reconsidered for writes. If the value
is zero, the node is excluded for the life of the client
--
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]
