This is an automated email from the ASF dual-hosted git repository.

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new d71859682b2 Move DatabaseChangedListenerAssistedPersistService to 
cluster-core module (#34484)
d71859682b2 is described below

commit d71859682b2c13965446e7c8e5a5ab81254396c2
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Sat Jan 25 14:54:13 2025 +0800

    Move DatabaseChangedListenerAssistedPersistService to cluster-core module 
(#34484)
---
 .../shardingsphere/mode/persist/PersistServiceFacade.java      | 10 +++-------
 .../global/DatabaseChangedListenerAssistedChangedHandler.java  |  4 ++--
 .../persist/service/ClusterMetaDataManagerPersistService.java  |  2 --
 .../DatabaseChangedListenerAssistedPersistService.java         |  2 +-
 .../persist/service}/DatabaseChangedListenerAssistedType.java  |  2 +-
 .../DatabaseChangedListenerAssistedChangedHandlerTest.java     |  2 +-
 .../service/ClusterMetaDataManagerPersistServiceTest.java      |  2 --
 .../DatabaseChangedListenerAssistedPersistServiceTest.java     |  2 +-
 8 files changed, 9 insertions(+), 17 deletions(-)

diff --git 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/persist/PersistServiceFacade.java
 
b/mode/core/src/main/java/org/apache/shardingsphere/mode/persist/PersistServiceFacade.java
index a8d68b7f44e..de5b3a24cad 100644
--- 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/persist/PersistServiceFacade.java
+++ 
b/mode/core/src/main/java/org/apache/shardingsphere/mode/persist/PersistServiceFacade.java
@@ -21,16 +21,15 @@ import lombok.Getter;
 import org.apache.shardingsphere.infra.config.mode.ModeConfiguration;
 import org.apache.shardingsphere.infra.instance.ComputeNodeInstance;
 import org.apache.shardingsphere.infra.spi.type.typed.TypedSPILoader;
-import org.apache.shardingsphere.mode.metadata.persist.MetaDataPersistService;
 import org.apache.shardingsphere.mode.metadata.MetaDataContextManager;
+import org.apache.shardingsphere.mode.metadata.persist.MetaDataPersistService;
 import 
org.apache.shardingsphere.mode.persist.service.MetaDataManagerPersistService;
 import org.apache.shardingsphere.mode.persist.service.PersistServiceBuilder;
 import org.apache.shardingsphere.mode.persist.service.ProcessPersistService;
+import org.apache.shardingsphere.mode.spi.repository.PersistRepository;
+import org.apache.shardingsphere.mode.state.cluster.ClusterStatePersistService;
 import org.apache.shardingsphere.mode.state.node.ComputeNodePersistService;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedPersistService;
 import 
org.apache.shardingsphere.mode.state.node.QualifiedDataSourceStatePersistService;
-import org.apache.shardingsphere.mode.state.cluster.ClusterStatePersistService;
-import org.apache.shardingsphere.mode.spi.repository.PersistRepository;
 
 /**
  * Persist service facade.
@@ -50,8 +49,6 @@ public final class PersistServiceFacade {
     
     private final ProcessPersistService processPersistService;
     
-    private final DatabaseChangedListenerAssistedPersistService 
databaseChangedListenerAssistedPersistService;
-    
     private final QualifiedDataSourceStatePersistService 
qualifiedDataSourceStatePersistService;
     
     public PersistServiceFacade(final PersistRepository repository, final 
ModeConfiguration modeConfig, final MetaDataContextManager 
metaDataContextManager) {
@@ -63,7 +60,6 @@ public final class PersistServiceFacade {
         PersistServiceBuilder persistServiceBuilder = 
TypedSPILoader.getService(PersistServiceBuilder.class, modeConfig.getType());
         metaDataManagerPersistService = 
persistServiceBuilder.buildMetaDataManagerPersistService(repository, 
metaDataContextManager);
         processPersistService = 
persistServiceBuilder.buildProcessPersistService(repository);
-        databaseChangedListenerAssistedPersistService = new 
DatabaseChangedListenerAssistedPersistService(repository);
     }
     
     /**
diff --git 
a/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandler.java
 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandler.java
index 0bf276b902b..b2ce8d22076 100644
--- 
a/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandler.java
+++ 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandler.java
@@ -27,8 +27,8 @@ import 
org.apache.shardingsphere.mode.metadata.refresher.statistics.StatisticsRe
 import 
org.apache.shardingsphere.mode.node.path.metadata.DatabaseMetaDataNodePath;
 import org.apache.shardingsphere.mode.node.path.state.StatesNodePath;
 import 
org.apache.shardingsphere.mode.repository.cluster.ClusterPersistRepository;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedPersistService;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedType;
+import 
org.apache.shardingsphere.mode.manager.cluster.persist.service.DatabaseChangedListenerAssistedPersistService;
+import 
org.apache.shardingsphere.mode.manager.cluster.persist.service.DatabaseChangedListenerAssistedType;
 
 import java.util.Arrays;
 import java.util.Collection;
diff --git 
a/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistService.java
 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistService.java
index 53194d37610..8ad3de399f2 100644
--- 
a/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistService.java
+++ 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistService.java
@@ -38,8 +38,6 @@ import 
org.apache.shardingsphere.mode.metadata.persist.config.database.DataSourc
 import 
org.apache.shardingsphere.mode.metadata.persist.metadata.DatabaseMetaDataPersistFacade;
 import 
org.apache.shardingsphere.mode.persist.service.MetaDataManagerPersistService;
 import org.apache.shardingsphere.mode.spi.repository.PersistRepository;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedPersistService;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedType;
 import org.apache.shardingsphere.single.config.SingleRuleConfiguration;
 import org.apache.shardingsphere.single.rule.SingleRule;
 
diff --git 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistService.java
 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistService.java
similarity index 96%
rename from 
mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistService.java
rename to 
mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistService.java
index eb421fabdae..8d75d30f23e 100644
--- 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistService.java
+++ 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistService.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.shardingsphere.mode.state.database;
+package org.apache.shardingsphere.mode.manager.cluster.persist.service;
 
 import lombok.RequiredArgsConstructor;
 import org.apache.shardingsphere.mode.node.path.state.StatesNodePath;
diff --git 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedType.java
 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedType.java
similarity index 93%
rename from 
mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedType.java
rename to 
mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedType.java
index 14560566d2d..7a29d5ba553 100644
--- 
a/mode/core/src/main/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedType.java
+++ 
b/mode/type/cluster/core/src/main/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedType.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.shardingsphere.mode.state.database;
+package org.apache.shardingsphere.mode.manager.cluster.persist.service;
 
 /**
  * Database changed listener assisted type.
diff --git 
a/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandlerTest.java
 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandlerTest.java
index e27b63e102b..61dfc816bc1 100644
--- 
a/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandlerTest.java
+++ 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/dispatch/handler/global/DatabaseChangedListenerAssistedChangedHandlerTest.java
@@ -63,7 +63,7 @@ class DatabaseChangedListenerAssistedChangedHandlerTest {
     @Test
     void assertHandleWithoutDatabase() {
         handler.handle(contextManager, new 
DataChangedEvent("/states/listener_assisted", "", Type.ADDED));
-        verify(contextManager.getPersistServiceFacade(), 
times(0)).getDatabaseChangedListenerAssistedPersistService();
+        verify(contextManager.getPersistServiceFacade(), 
times(0)).getRepository();
     }
     
     @Test
diff --git 
a/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistServiceTest.java
 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistServiceTest.java
index bad508d4007..2a76832d7ee 100644
--- 
a/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistServiceTest.java
+++ 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/ClusterMetaDataManagerPersistServiceTest.java
@@ -25,8 +25,6 @@ import 
org.apache.shardingsphere.infra.metadata.database.schema.model.ShardingSp
 import org.apache.shardingsphere.mode.metadata.MetaDataContextManager;
 import org.apache.shardingsphere.mode.metadata.persist.MetaDataPersistService;
 import org.apache.shardingsphere.mode.spi.repository.PersistRepository;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedPersistService;
-import 
org.apache.shardingsphere.mode.state.database.DatabaseChangedListenerAssistedType;
 import org.apache.shardingsphere.single.config.SingleRuleConfiguration;
 import org.apache.shardingsphere.single.rule.SingleRule;
 import org.junit.jupiter.api.BeforeEach;
diff --git 
a/mode/core/src/test/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistServiceTest.java
 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistServiceTest.java
similarity index 96%
rename from 
mode/core/src/test/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistServiceTest.java
rename to 
mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistServiceTest.java
index 6f8a701880f..fe48fac4e5e 100644
--- 
a/mode/core/src/test/java/org/apache/shardingsphere/mode/state/database/DatabaseChangedListenerAssistedPersistServiceTest.java
+++ 
b/mode/type/cluster/core/src/test/java/org/apache/shardingsphere/mode/manager/cluster/persist/service/DatabaseChangedListenerAssistedPersistServiceTest.java
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.shardingsphere.mode.state.database;
+package org.apache.shardingsphere.mode.manager.cluster.persist.service;
 
 import org.apache.shardingsphere.mode.spi.repository.PersistRepository;
 import org.junit.jupiter.api.Test;

Reply via email to