smengcl commented on a change in pull request #1552:
URL: https://github.com/apache/ozone/pull/1552#discussion_r519836931
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/states/endpoint/HeartbeatEndpointTask.java
##########
@@ -147,8 +147,12 @@ public void setDatanodeDetailsProto(DatanodeDetailsProto
rpcEndpoint.setLastSuccessfulHeartbeat(ZonedDateTime.now());
rpcEndpoint.zeroMissedCount();
} catch (IOException ex) {
- // put back the reports which failed to be sent
- putBackReports(requestBuilder);
+ // don't resend reports to recon as it could be down for days
+ // DN is expected to work fine without recon and not go OOM
Review comment:
Also I believe container actions and pipeline actions **are** put back
on exception. cmiiw.
cc @nandakumar131 @lokeshj1703
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
