bshashikant commented on a change in pull request #54: HDDS-2281.
ContainerStateMachine#handleWriteChunk should ignore close container exception
URL: https://github.com/apache/hadoop-ozone/pull/54#discussion_r337406349
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/transport/server/ratis/ContainerStateMachine.java
##########
@@ -460,6 +460,10 @@ private ExecutorService getCommandExecutor(
LOG.error(gid + ": writeChunk writeStateMachineData failed:
blockId"
+ write.getBlockID() + " logIndex " + entryIndex + " chunkName
"
+ write.getChunkData().getChunkName() + e);
+ metrics.incNumWriteDataFails();
+ // write chunks go in parallel. It's possible that one write chunk
+ // see the stateMachine is marked unhealthy by other parallel
thread.
+ stateMachineHealthy.set(false);
Review comment:
> I know this patch is merged, but I have no way of understanding what this
means -- so I appreciate some comments or feedback that explains what happens
here?
Updated the description to add clarity to this.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org
