Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Improve LocalProxyTransaction.doExists()
[controller.git]
/
opendaylight
/
md-sal
/
sal-distributed-datastore
/
src
/
main
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
datastore
/
shardmanager
/
ShardPeerAddressResolver.java
diff --git
a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardmanager/ShardPeerAddressResolver.java
b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardmanager/ShardPeerAddressResolver.java
index 468d650d4da2225412e12be588e0a8003e5f7b51..6c33652bb1a6cde93cad77ec48adbe4068289060 100644
(file)
--- a/
opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardmanager/ShardPeerAddressResolver.java
+++ b/
opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/shardmanager/ShardPeerAddressResolver.java
@@
-7,8
+7,10
@@
*/
package org.opendaylight.controller.cluster.datastore.shardmanager;
*/
package org.opendaylight.controller.cluster.datastore.shardmanager;
+import static java.util.Objects.requireNonNull;
+
import akka.actor.Address;
import akka.actor.Address;
-import
com.google.common.base.Preconditions
;
+import
akka.actor.AddressFromURIString
;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Map;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Map;
@@
-34,17
+36,17
@@
class ShardPeerAddressResolver implements PeerAddressResolver {
private final String shardManagerType;
private final MemberName localMemberName;
private final String shardManagerType;
private final MemberName localMemberName;
- ShardPeerAddressResolver(
String shardManagerType,
MemberName localMemberName) {
+ ShardPeerAddressResolver(
final String shardManagerType, final
MemberName localMemberName) {
this.shardManagerIdentifier = ShardManagerIdentifier.builder().type(shardManagerType).build().toString();
this.shardManagerType = shardManagerType;
this.shardManagerIdentifier = ShardManagerIdentifier.builder().type(shardManagerType).build().toString();
this.shardManagerType = shardManagerType;
- this.localMemberName =
Preconditions.checkNot
Null(localMemberName);
+ this.localMemberName =
requireNon
Null(localMemberName);
}
}
- void addPeerAddress(
MemberName memberName,
Address address) {
+ void addPeerAddress(
final MemberName memberName, final
Address address) {
memberNameToAddress.put(memberName, address);
}
memberNameToAddress.put(memberName, address);
}
- void removePeerAddress(MemberName memberName) {
+ void removePeerAddress(
final
MemberName memberName) {
memberNameToAddress.remove(memberName);
}
memberNameToAddress.remove(memberName);
}
@@
-52,7
+54,7
@@
class ShardPeerAddressResolver implements PeerAddressResolver {
return this.memberNameToAddress.keySet();
}
return this.memberNameToAddress.keySet();
}
- Address getPeerAddress(MemberName memberName) {
+ Address getPeerAddress(
final
MemberName memberName) {
return memberNameToAddress.get(memberName);
}
return memberNameToAddress.get(memberName);
}
@@
-67,11
+69,11
@@
class ShardPeerAddressResolver implements PeerAddressResolver {
return peerAddresses;
}
return peerAddresses;
}
- ShardIdentifier getShardIdentifier(
MemberName memberName,
String shardName) {
+ ShardIdentifier getShardIdentifier(
final MemberName memberName, final
String shardName) {
return ShardIdentifier.create(shardName, memberName, shardManagerType);
}
return ShardIdentifier.create(shardName, memberName, shardManagerType);
}
- String getShardActorAddress(
String shardName,
MemberName memberName) {
+ String getShardActorAddress(
final String shardName, final
MemberName memberName) {
Address memberAddress = memberNameToAddress.get(memberName);
if (memberAddress != null) {
return getShardManagerActorPathBuilder(memberAddress).append("/").append(
Address memberAddress = memberNameToAddress.get(memberName);
if (memberAddress != null) {
return getShardManagerActorPathBuilder(memberAddress).append("/").append(
@@
-81,12
+83,12
@@
class ShardPeerAddressResolver implements PeerAddressResolver {
return null;
}
return null;
}
- StringBuilder getShardManagerActorPathBuilder(Address address) {
+ StringBuilder getShardManagerActorPathBuilder(
final
Address address) {
return new StringBuilder().append(address.toString()).append("/user/").append(shardManagerIdentifier);
}
@Override
return new StringBuilder().append(address.toString()).append("/user/").append(shardManagerIdentifier);
}
@Override
- public String resolve(String peerId) {
+ public String resolve(
final
String peerId) {
if (peerId == null) {
return null;
}
if (peerId == null) {
return null;
}
@@
-94,4
+96,10
@@
class ShardPeerAddressResolver implements PeerAddressResolver {
ShardIdentifier shardId = ShardIdentifier.fromShardIdString(peerId);
return getShardActorAddress(shardId.getShardName(), shardId.getMemberName());
}
ShardIdentifier shardId = ShardIdentifier.fromShardIdString(peerId);
return getShardActorAddress(shardId.getShardName(), shardId.getMemberName());
}
+
+ @Override
+ public void setResolved(final String peerId, final String address) {
+ memberNameToAddress.put(ShardIdentifier.fromShardIdString(peerId).getMemberName(),
+ AddressFromURIString.parse(address));
+ }
}
}