Changed type of term number and sequence number to long.
Change-Id: If90663fa9567988752f8c4af17daeff904865ab7
Showing
5 changed files
with
17 additions
and
17 deletions
| ... | @@ -24,14 +24,14 @@ import com.google.common.base.MoreObjects; | ... | @@ -24,14 +24,14 @@ import com.google.common.base.MoreObjects; |
| 24 | public final class MastershipTerm { | 24 | public final class MastershipTerm { |
| 25 | 25 | ||
| 26 | private final NodeId master; | 26 | private final NodeId master; |
| 27 | - private final int termNumber; | 27 | + private final long termNumber; |
| 28 | 28 | ||
| 29 | - private MastershipTerm(NodeId master, int term) { | 29 | + private MastershipTerm(NodeId master, long term) { |
| 30 | this.master = master; | 30 | this.master = master; |
| 31 | this.termNumber = term; | 31 | this.termNumber = term; |
| 32 | } | 32 | } |
| 33 | 33 | ||
| 34 | - public static MastershipTerm of(NodeId master, int term) { | 34 | + public static MastershipTerm of(NodeId master, long term) { |
| 35 | return new MastershipTerm(master, term); | 35 | return new MastershipTerm(master, term); |
| 36 | } | 36 | } |
| 37 | 37 | ||
| ... | @@ -39,7 +39,7 @@ public final class MastershipTerm { | ... | @@ -39,7 +39,7 @@ public final class MastershipTerm { |
| 39 | return master; | 39 | return master; |
| 40 | } | 40 | } |
| 41 | 41 | ||
| 42 | - public int termNumber() { | 42 | + public long termNumber() { |
| 43 | return termNumber; | 43 | return termNumber; |
| 44 | } | 44 | } |
| 45 | 45 | ... | ... |
| ... | @@ -19,7 +19,7 @@ import static org.slf4j.LoggerFactory.getLogger; | ... | @@ -19,7 +19,7 @@ import static org.slf4j.LoggerFactory.getLogger; |
| 19 | 19 | ||
| 20 | import java.util.concurrent.ConcurrentHashMap; | 20 | import java.util.concurrent.ConcurrentHashMap; |
| 21 | import java.util.concurrent.ConcurrentMap; | 21 | import java.util.concurrent.ConcurrentMap; |
| 22 | -import java.util.concurrent.atomic.AtomicInteger; | 22 | +import java.util.concurrent.atomic.AtomicLong; |
| 23 | 23 | ||
| 24 | import org.apache.felix.scr.annotations.Activate; | 24 | import org.apache.felix.scr.annotations.Activate; |
| 25 | import org.apache.felix.scr.annotations.Component; | 25 | import org.apache.felix.scr.annotations.Component; |
| ... | @@ -43,7 +43,7 @@ public class DeviceClockManager implements DeviceClockService, DeviceClockProvid | ... | @@ -43,7 +43,7 @@ public class DeviceClockManager implements DeviceClockService, DeviceClockProvid |
| 43 | private final Logger log = getLogger(getClass()); | 43 | private final Logger log = getLogger(getClass()); |
| 44 | 44 | ||
| 45 | // TODO: Implement per device ticker that is reset to 0 at the beginning of a new term. | 45 | // TODO: Implement per device ticker that is reset to 0 at the beginning of a new term. |
| 46 | - private final AtomicInteger ticker = new AtomicInteger(0); | 46 | + private final AtomicLong ticker = new AtomicLong(0); |
| 47 | private ConcurrentMap<DeviceId, MastershipTerm> deviceMastershipTerms = new ConcurrentHashMap<>(); | 47 | private ConcurrentMap<DeviceId, MastershipTerm> deviceMastershipTerms = new ConcurrentHashMap<>(); |
| 48 | 48 | ||
| 49 | @Activate | 49 | @Activate | ... | ... |
| ... | @@ -34,8 +34,8 @@ import com.google.common.collect.ComparisonChain; | ... | @@ -34,8 +34,8 @@ import com.google.common.collect.ComparisonChain; |
| 34 | */ | 34 | */ |
| 35 | public final class MastershipBasedTimestamp implements Timestamp { | 35 | public final class MastershipBasedTimestamp implements Timestamp { |
| 36 | 36 | ||
| 37 | - private final int termNumber; | 37 | + private final long termNumber; |
| 38 | - private final int sequenceNumber; | 38 | + private final long sequenceNumber; |
| 39 | 39 | ||
| 40 | /** | 40 | /** |
| 41 | * Default version tuple. | 41 | * Default version tuple. |
| ... | @@ -43,7 +43,7 @@ public final class MastershipBasedTimestamp implements Timestamp { | ... | @@ -43,7 +43,7 @@ public final class MastershipBasedTimestamp implements Timestamp { |
| 43 | * @param termNumber the mastership termNumber | 43 | * @param termNumber the mastership termNumber |
| 44 | * @param sequenceNumber the sequenceNumber number within the termNumber | 44 | * @param sequenceNumber the sequenceNumber number within the termNumber |
| 45 | */ | 45 | */ |
| 46 | - public MastershipBasedTimestamp(int termNumber, int sequenceNumber) { | 46 | + public MastershipBasedTimestamp(long termNumber, long sequenceNumber) { |
| 47 | this.termNumber = termNumber; | 47 | this.termNumber = termNumber; |
| 48 | this.sequenceNumber = sequenceNumber; | 48 | this.sequenceNumber = sequenceNumber; |
| 49 | } | 49 | } |
| ... | @@ -91,7 +91,7 @@ public final class MastershipBasedTimestamp implements Timestamp { | ... | @@ -91,7 +91,7 @@ public final class MastershipBasedTimestamp implements Timestamp { |
| 91 | * | 91 | * |
| 92 | * @return termNumber | 92 | * @return termNumber |
| 93 | */ | 93 | */ |
| 94 | - public int termNumber() { | 94 | + public long termNumber() { |
| 95 | return termNumber; | 95 | return termNumber; |
| 96 | } | 96 | } |
| 97 | 97 | ||
| ... | @@ -100,7 +100,7 @@ public final class MastershipBasedTimestamp implements Timestamp { | ... | @@ -100,7 +100,7 @@ public final class MastershipBasedTimestamp implements Timestamp { |
| 100 | * | 100 | * |
| 101 | * @return sequenceNumber | 101 | * @return sequenceNumber |
| 102 | */ | 102 | */ |
| 103 | - public int sequenceNumber() { | 103 | + public long sequenceNumber() { |
| 104 | return sequenceNumber; | 104 | return sequenceNumber; |
| 105 | } | 105 | } |
| 106 | 106 | ... | ... |
| ... | @@ -38,14 +38,14 @@ public class MastershipBasedTimestampSerializer extends Serializer<MastershipBas | ... | @@ -38,14 +38,14 @@ public class MastershipBasedTimestampSerializer extends Serializer<MastershipBas |
| 38 | 38 | ||
| 39 | @Override | 39 | @Override |
| 40 | public void write(Kryo kryo, Output output, MastershipBasedTimestamp object) { | 40 | public void write(Kryo kryo, Output output, MastershipBasedTimestamp object) { |
| 41 | - output.writeInt(object.termNumber()); | 41 | + output.writeLong(object.termNumber()); |
| 42 | - output.writeInt(object.sequenceNumber()); | 42 | + output.writeLong(object.sequenceNumber()); |
| 43 | } | 43 | } |
| 44 | 44 | ||
| 45 | @Override | 45 | @Override |
| 46 | public MastershipBasedTimestamp read(Kryo kryo, Input input, Class<MastershipBasedTimestamp> type) { | 46 | public MastershipBasedTimestamp read(Kryo kryo, Input input, Class<MastershipBasedTimestamp> type) { |
| 47 | - final int term = input.readInt(); | 47 | + final long term = input.readLong(); |
| 48 | - final int sequence = input.readInt(); | 48 | + final long sequence = input.readLong(); |
| 49 | return new MastershipBasedTimestamp(term, sequence); | 49 | return new MastershipBasedTimestamp(term, sequence); |
| 50 | } | 50 | } |
| 51 | } | 51 | } | ... | ... |
core/store/serializers/src/main/java/org/onosproject/store/serializers/MastershipTermSerializer.java
| ... | @@ -39,13 +39,13 @@ public class MastershipTermSerializer extends Serializer<MastershipTerm> { | ... | @@ -39,13 +39,13 @@ public class MastershipTermSerializer extends Serializer<MastershipTerm> { |
| 39 | @Override | 39 | @Override |
| 40 | public MastershipTerm read(Kryo kryo, Input input, Class<MastershipTerm> type) { | 40 | public MastershipTerm read(Kryo kryo, Input input, Class<MastershipTerm> type) { |
| 41 | final NodeId node = (NodeId) kryo.readClassAndObject(input); | 41 | final NodeId node = (NodeId) kryo.readClassAndObject(input); |
| 42 | - final int term = input.readInt(); | 42 | + final long term = input.readLong(); |
| 43 | return MastershipTerm.of(node, term); | 43 | return MastershipTerm.of(node, term); |
| 44 | } | 44 | } |
| 45 | 45 | ||
| 46 | @Override | 46 | @Override |
| 47 | public void write(Kryo kryo, Output output, MastershipTerm object) { | 47 | public void write(Kryo kryo, Output output, MastershipTerm object) { |
| 48 | kryo.writeClassAndObject(output, object.master()); | 48 | kryo.writeClassAndObject(output, object.master()); |
| 49 | - output.writeInt(object.termNumber()); | 49 | + output.writeLong(object.termNumber()); |
| 50 | } | 50 | } |
| 51 | } | 51 | } | ... | ... |
-
Please register or login to post a comment