diff options
author | Bjørn Christian Seime <bjorncs@oath.com> | 2018-01-25 17:38:21 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@oath.com> | 2018-01-25 17:38:21 +0100 |
commit | b1ecc4ad41a22ee355a1e2ad6b68654e14ad9476 (patch) | |
tree | e982f248fd419300c5fdc3f52f450389fe66abc7 /container-core/src/test/java/com/yahoo/container/jdisc/state | |
parent | 80d1fa0e0994c29bcd1cecf944191e99e42da123 (diff) |
Use AtomicLong instead of volatile long
Diffstat (limited to 'container-core/src/test/java/com/yahoo/container/jdisc/state')
-rw-r--r-- | container-core/src/test/java/com/yahoo/container/jdisc/state/StateHandlerTest.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/container-core/src/test/java/com/yahoo/container/jdisc/state/StateHandlerTest.java b/container-core/src/test/java/com/yahoo/container/jdisc/state/StateHandlerTest.java index 100650d43bd..5d8f885f8d0 100644 --- a/container-core/src/test/java/com/yahoo/container/jdisc/state/StateHandlerTest.java +++ b/container-core/src/test/java/com/yahoo/container/jdisc/state/StateHandlerTest.java @@ -30,6 +30,7 @@ import java.util.HashMap; import java.util.Map; import java.util.TreeMap; import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicLong; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; @@ -46,7 +47,7 @@ public class StateHandlerTest { private TestDriver driver; private StateMonitor monitor; private Metric metric; - private volatile long currentTimeMillis = 0; + private final AtomicLong currentTimeMillis = new AtomicLong(0); @Before public void startTestDriver() { @@ -58,7 +59,7 @@ public class StateHandlerTest { @Override public long currentTimeMillis() { - return currentTimeMillis; + return currentTimeMillis.get(); } }); } @@ -400,7 +401,7 @@ public class StateHandlerTest { } private void incrementCurrentTime(long val) { - currentTimeMillis += val; + currentTimeMillis.addAndGet(val); monitor.checkTime(); } |