X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2Fshardstrategy%2FShardStrategyFactory.java;h=e63ab9744514cf2fd94e4d822df3fc020ce52058;hb=a6af137c30470b86d4bc624d4c48cb686495a182;hp=8ca615003872116f84138bba22ddca177da4e28c;hpb=18ecb07132281d3152a49baf50e8c020403588f9;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardstrategy/ShardStrategyFactory.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardstrategy/ShardStrategyFactory.java index 8ca6150038..e63ab97445 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardstrategy/ShardStrategyFactory.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardstrategy/ShardStrategyFactory.java @@ -10,33 +10,44 @@ package org.opendaylight.controller.cluster.datastore.shardstrategy; import com.google.common.base.Preconditions; import org.opendaylight.controller.cluster.datastore.config.Configuration; +import org.opendaylight.mdsal.common.api.LogicalDatastoreType; +import org.opendaylight.mdsal.dom.api.DOMDataTreeIdentifier; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; public class ShardStrategyFactory { private static final String UNKNOWN_MODULE_NAME = "unknown"; private final Configuration configuration; + private final LogicalDatastoreType logicalStoreType; - public ShardStrategyFactory(final Configuration configuration) { + public ShardStrategyFactory(final Configuration configuration, final LogicalDatastoreType logicalStoreType) { Preconditions.checkState(configuration != null, "configuration should not be missing"); this.configuration = configuration; + this.logicalStoreType = Preconditions.checkNotNull(logicalStoreType); } public ShardStrategy getStrategy(final YangInstanceIdentifier path) { Preconditions.checkNotNull(path, "path should not be null"); - String moduleName = getModuleName(path); - ShardStrategy shardStrategy = configuration.getStrategyForModule(moduleName); + // try with the legacy module based shard mapping + final String moduleName = getModuleName(path); + final ShardStrategy shardStrategy = configuration.getStrategyForModule(moduleName); if (shardStrategy == null) { - return DefaultShardStrategy.getInstance(); + // retry with prefix based sharding + final ShardStrategy strategyForPrefix = + configuration.getStrategyForPrefix(new DOMDataTreeIdentifier(logicalStoreType, path)); + if (strategyForPrefix == null) { + return DefaultShardStrategy.getInstance(); + } + return strategyForPrefix; } return shardStrategy; } - public static ShardStrategy newShardStrategyInstance(String moduleName, String strategyName, - Configuration configuration) { - if(ModuleShardStrategy.NAME.equals(strategyName)){ + public static ShardStrategy newShardStrategyInstance(final String moduleName, final String strategyName, + final Configuration configuration) { + if (ModuleShardStrategy.NAME.equals(strategyName)) { return new ModuleShardStrategy(moduleName, configuration); } @@ -48,7 +59,7 @@ public class ShardStrategyFactory { return UNKNOWN_MODULE_NAME; } - String namespace = path.getPathArguments().iterator().next().getNodeType().getNamespace().toASCIIString(); + String namespace = path.getPathArguments().get(0).getNodeType().getNamespace().toASCIIString(); String moduleName = configuration.getModuleNameFromNameSpace(namespace); return moduleName != null ? moduleName : UNKNOWN_MODULE_NAME; }