bharatviswa504 commented on a change in pull request #846: HDDS-1555. Disable
install snapshot for ContainerStateMachine.
URL: https://github.com/apache/hadoop/pull/846#discussion_r287045256
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/transport/server/ratis/XceiverServerRatis.java
##########
@@ -597,4 +603,12 @@ void handleNodeSlowness(RaftGroup group, RoleInfoProto
roleInfoProto) {
void handleNoLeader(RaftGroup group, RoleInfoProto roleInfoProto) {
handlePipelineFailure(group.getGroupId(), roleInfoProto);
}
+
+ void handleInstallSnapshotFromLeader(RaftGroup group,
+ RoleInfoProto roleInfoProto, TermIndex firstTermIndexInLog) {
+ LOG.warn("Install snapshot notification received from Leader with " +
+ "termIndex : " + firstTermIndexInLog +
+ ", terminating pipeline " + group.getGroupId());
+ handlePipelineFailure(group.getGroupId(), roleInfoProto);
Review comment:
I am trying to understand this change, Jira says disable install snapshot
for ContainerStateMachine.
I see that setSnapShotEnabled is set to false.
But what is the reason for handleInstallSnapShotFromLeader to call
handlePipelineFailure.
Sorry, I don't have complete context of this, can you provide some
explanation of this change, and why it is done, from which I can understand
better.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
