X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2Fshardstrategy%2FShardStrategyFactory.java;h=724499477ebde6a5b03fbba4a72d3c14564a6886;hp=f4ab8fab6f768e84bd48c15ebc83e862315e09eb;hb=c1336f9b497bc6867536a24f629c3f0b002ccb2f;hpb=11e9ade9af527aba7faeb633d3c9c7552fd09d2d 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 f4ab8fab6f..724499477e 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 @@ -8,52 +8,59 @@ package org.opendaylight.controller.cluster.datastore.shardstrategy; -import com.google.common.base.Optional; import com.google.common.base.Preconditions; -import org.opendaylight.controller.cluster.datastore.Configuration; +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; -import java.util.Map; -import java.util.concurrent.ConcurrentHashMap; - public class ShardStrategyFactory { - private static Map moduleNameToStrategyMap = - new ConcurrentHashMap(); - private static final String UNKNOWN_MODULE_NAME = "unknown"; - private static Configuration configuration; + private final Configuration configuration; + private final LogicalDatastoreType logicalStoreType; - public static void setConfiguration(Configuration configuration){ - ShardStrategyFactory.configuration = configuration; - moduleNameToStrategyMap = configuration.getModuleNameToShardStrategyMap(); + public ShardStrategyFactory(final Configuration configuration, LogicalDatastoreType logicalStoreType) { + Preconditions.checkState(configuration != null, "configuration should not be missing"); + this.configuration = configuration; + this.logicalStoreType = Preconditions.checkNotNull(logicalStoreType); } - public static ShardStrategy getStrategy(YangInstanceIdentifier path) { - Preconditions.checkState(configuration != null, "configuration should not be missing"); + public ShardStrategy getStrategy(final YangInstanceIdentifier path) { Preconditions.checkNotNull(path, "path should not be null"); - - String moduleName = getModuleName(path); - ShardStrategy shardStrategy = moduleNameToStrategyMap.get(moduleName); + // try with the legacy module based shard mapping + final String moduleName = getModuleName(path); + final ShardStrategy shardStrategy = configuration.getStrategyForModule(moduleName); if (shardStrategy == null) { - return new DefaultShardStrategy(); + // 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)) { + return new ModuleShardStrategy(moduleName, configuration); + } - private static String getModuleName(YangInstanceIdentifier path) { - String namespace = path.getPathArguments().iterator().next().getNodeType().getNamespace().toASCIIString(); - - Optional optional = - configuration.getModuleNameFromNameSpace(namespace); + return DefaultShardStrategy.getInstance(); + } - if(!optional.isPresent()){ + private String getModuleName(final YangInstanceIdentifier path) { + if (path.isEmpty()) { return UNKNOWN_MODULE_NAME; } - return optional.get(); + String namespace = path.getPathArguments().iterator().next().getNodeType().getNamespace().toASCIIString(); + String moduleName = configuration.getModuleNameFromNameSpace(namespace); + return moduleName != null ? moduleName : UNKNOWN_MODULE_NAME; } }