errose28 commented on code in PR #9779:
URL: https://github.com/apache/ozone/pull/9779#discussion_r2848446889
##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/upgrade/FinalizationStateManagerImpl.java:
##########
@@ -72,27 +71,6 @@ private void publishCheckpoint(FinalizationCheckpoint
checkpoint) {
// Move the upgrade status according to this checkpoint. This is sent
// back to the client if they query for the current upgrade status.
versionManager.setUpgradeState(checkpoint.getStatus());
-
- // Check whether this checkpoint change requires us to move node state.
- // If this is necessary, it must be done before unfreezing pipeline
- // creation to make sure nodes are not added to pipelines based on
- // outdated layout information.
- // This operation is not idempotent.
- if (checkpoint == FinalizationCheckpoint.MLV_EQUALS_SLV) {
- upgradeContext.getNodeManager().forceNodesToHealthyReadOnly();
Review Comment:
This will be handled by HDDS-14671
--
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]
