umamaheswararao commented on a change in pull request #2849:
URL: https://github.com/apache/ozone/pull/2849#discussion_r758671284
##########
File path:
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/io/ECBlockOutputStreamEntry.java
##########
@@ -267,7 +269,11 @@ void executePutBlock() throws IOException {
if (stream == null) {
continue;
}
- stream.executePutBlock(false, true);
+ try {
+ stream.executePutBlock(false, true);
+ } catch (Exception e) {
+ stream.setIoException(e);
Review comment:
This is a good point @fapifta . Actually currently we are not retrying
in handleStripeFailure. There is a JIRA(HDDS-6036) for it. The idea is to make
that method retry and in that retry path we have to do the same exception
handling and loop. Currently I am thinking to retry some bounded number of
times. It should bounded as it will request new blockgroup on each retry.
--
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]
