Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
BUG-7594: Rework sal-remoterpc-connector messages
[controller.git]
/
opendaylight
/
md-sal
/
sal-remoterpc-connector
/
src
/
main
/
java
/
org
/
opendaylight
/
controller
/
remote
/
rpc
/
registry
/
gossip
/
BucketImpl.java
diff --git
a/opendaylight/md-sal/sal-remoterpc-connector/src/main/java/org/opendaylight/controller/remote/rpc/registry/gossip/BucketImpl.java
b/opendaylight/md-sal/sal-remoterpc-connector/src/main/java/org/opendaylight/controller/remote/rpc/registry/gossip/BucketImpl.java
index 4dac456dc426d3493c8988355f40a79ca1df97ab..daf945289c42b65b6adc1d0c8d31529749a378b5 100644
(file)
--- a/
opendaylight/md-sal/sal-remoterpc-connector/src/main/java/org/opendaylight/controller/remote/rpc/registry/gossip/BucketImpl.java
+++ b/
opendaylight/md-sal/sal-remoterpc-connector/src/main/java/org/opendaylight/controller/remote/rpc/registry/gossip/BucketImpl.java
@@
-7,37
+7,40
@@
*/
package org.opendaylight.controller.remote.rpc.registry.gossip;
*/
package org.opendaylight.controller.remote.rpc.registry.gossip;
+import com.google.common.base.Preconditions;
+import com.google.common.base.Verify;
import java.io.Serializable;
import java.io.Serializable;
-public class BucketImpl<T extends Copier<T>> implements Bucket<T>, Serializable {
+final class BucketImpl<T extends BucketData<T>> implements Bucket<T>, Serializable {
+ private static final long serialVersionUID = 1L;
- private
Long version = System.currentTimeMillis()
;
+ private
final long version
;
- private T data;
+ // Guaranteed to be non-null
+ private final T data;
+
+ BucketImpl(final long version, final T data) {
+ this.version = version;
+ this.data = Preconditions.checkNotNull(data);
+ }
@Override
@Override
- public
L
ong getVersion() {
+ public
l
ong getVersion() {
return version;
}
@Override
public T getData() {
return version;
}
@Override
public T getData() {
- if (this.data == null)
- return null;
-
- return data.copy();
- }
-
- public void setData(T data){
- this.version = System.currentTimeMillis()+1;
- this.data = data;
+ return data;
}
@Override
public String toString() {
}
@Override
public String toString() {
- return "BucketImpl{" +
- "version=" + version +
- ", data=" + data +
- '}';
+ return "BucketImpl{" + "version=" + version + ", data=" + data + '}';
+ }
+
+ private Object readResolve() {
+ Verify.verifyNotNull(data);
+ return this;
}
}
}
}