You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "Peter Lee (Jira)" <ji...@apache.org> on 2025/04/22 08:26:00 UTC

[jira] [Updated] (HDDS-12881) Emits START_REPLICATION for ReplicationManager in SCMSafeModeManager#exitSafeMode

     [ https://issues.apache.org/jira/browse/HDDS-12881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Peter Lee updated HDDS-12881:
-----------------------------
    Description: 
Just notice that we the comment of SCMSafeModeManager#exitSafeMode says the safemode manager should tell the replicationManager that it can start to do the replication check. But I dont think we already have this.

[https://github.com/apache/ozone/pull/8311/files#diff-1ffe039aa52e2f283e868be3a0d68f09a3bd4c492b283333eb3f8a4b06f0fe51L220-L228]

 

I believe making the [ReplicationManagerEventHandler|https://github.com/apache/ozone/blob/master/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ReplicationManagerEventHandler.java] introduced in [https://github.com/apache/ozone/pull/7997] to also listen to {{START_REPLICATION}} event is enough to have the same effect.

  was:
Just notice that we the comment of SCMSafeModeManager#exitSafeMode says the safemode manager should tell the replicationManager that it can start to do the replication check. But I dont think we already have this.

[https://github.com/apache/ozone/pull/8311/files#diff-1ffe039aa52e2f283e868be3a0d68f09a3bd4c492b283333eb3f8a4b06f0fe51L220-L228]

 

I believe use the [ReplicationManagerEventHandler|https://github.com/apache/ozone/blob/master/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ReplicationManagerEventHandler.java] introduced in [https://github.com/apache/ozone/pull/7997] to also listen to {{START_REPLICATION}} event is enough to have the same effect.


> Emits START_REPLICATION for ReplicationManager in SCMSafeModeManager#exitSafeMode
> ---------------------------------------------------------------------------------
>
>                 Key: HDDS-12881
>                 URL: https://issues.apache.org/jira/browse/HDDS-12881
>             Project: Apache Ozone
>          Issue Type: Bug
>            Reporter: Peter Lee
>            Assignee: Peter Lee
>            Priority: Major
>
> Just notice that we the comment of SCMSafeModeManager#exitSafeMode says the safemode manager should tell the replicationManager that it can start to do the replication check. But I dont think we already have this.
> [https://github.com/apache/ozone/pull/8311/files#diff-1ffe039aa52e2f283e868be3a0d68f09a3bd4c492b283333eb3f8a4b06f0fe51L220-L228]
>  
> I believe making the [ReplicationManagerEventHandler|https://github.com/apache/ozone/blob/master/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ReplicationManagerEventHandler.java] introduced in [https://github.com/apache/ozone/pull/7997] to also listen to {{START_REPLICATION}} event is enough to have the same effect.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org