umamaheswararao commented on a change in pull request #2910:
URL: https://github.com/apache/ozone/pull/2910#discussion_r769107323
##########
File path:
hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/OzoneClientConfig.java
##########
@@ -124,6 +124,13 @@
tags = ConfigTag.CLIENT)
private boolean checksumVerify = true;
+ @Config(key = "max.ec.stripe.write.retries.on.failures",
Review comment:
I wanted to separate this from general write retries. In this case,
specifically we do retries at stripe boundary and if we can write that stripe
after trying this configured times we will give up the write completely. Where
as the regular write.retries(Mostly in non EC case) are per block level and
after it tried that many times,I think it would just give up that block write
and move on with new block allocations. So, I feel we can keep little high
number to make sure we will not give up too early. Based on the practice, we
may have to introduce incremental waits between retries to slow down that
unnecessary block allocations from OM.
--
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]
