aboutsummaryrefslogtreecommitdiffstats
path: root/zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@verizonmedia.com>2020-10-05 15:58:54 +0200
committerHåkon Hallingstad <hakon@verizonmedia.com>2020-10-05 15:58:54 +0200
commit967bff7bcfb043dbb5a926b7f61e68c8971759df (patch)
tree17fe62c79d51ece23e9c9daa7af82edfdecf10c7 /zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java
parente925ef8b0a33ed0e67e09ca9320b386339e08cea (diff)
Make richer latency stats
Makes a LatencyStats which provides some useful metrics, best explained there and in LatencyMetrics. This includes latency metrics, the "QPS" (e.g. the number of acquire() per second), and load metrics. Unfortunately I had to move from atomics to synchronized to accomplish this, but I see no other way.
Diffstat (limited to 'zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java')
-rw-r--r--zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java7
1 files changed, 0 insertions, 7 deletions
diff --git a/zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java b/zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java
index a7715eb9756..d673655c798 100644
--- a/zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java
+++ b/zkfacade/src/test/java/com/yahoo/vespa/curator/stats/LockTest.java
@@ -86,9 +86,6 @@ public class LockTest {
assertEquals(expected.getAndResetAcquireSucceededCount(), actual.getAndResetAcquireSucceededCount());
assertEquals(expected.getAndResetReleaseCount(), actual.getAndResetReleaseCount());
assertEquals(expected.getAndResetReleaseFailedCount(), actual.getAndResetReleaseFailedCount());
-
- assertEquals(expected.getAcquiringNow(), actual.getAcquiringNow());
- assertEquals(expected.getLockedNow(), actual.getLockedNow());
}
@Test
@@ -121,7 +118,6 @@ public class LockTest {
expectedMetrics.setCumulativeAcquireCount(1);
expectedMetrics.setAcquireSucceededCount(1);
expectedMetrics.setCumulativeAcquireSucceededCount(1);
- expectedMetrics.setLockedNow(1);
assertLockMetrics(expectedMetrics);
// reenter
@@ -131,7 +127,6 @@ public class LockTest {
expectedMetrics.setCumulativeAcquireCount(2);
expectedMetrics.setAcquireSucceededCount(1); // reset to 0 above, +1
expectedMetrics.setCumulativeAcquireSucceededCount(2);
- expectedMetrics.setLockedNow(2);
assertLockMetrics(expectedMetrics);
// inner-most closes
@@ -140,14 +135,12 @@ public class LockTest {
expectedMetrics.setAcquireSucceededCount(0); // reset to 0 above
expectedMetrics.setReleaseCount(1);
expectedMetrics.setCumulativeReleaseCount(1);
- expectedMetrics.setLockedNow(1);
assertLockMetrics(expectedMetrics);
// outer-most closes
lock.close();
expectedMetrics.setReleaseCount(1); // reset to 0 above, +1
expectedMetrics.setCumulativeReleaseCount(2);
- expectedMetrics.setLockedNow(0);
assertLockMetrics(expectedMetrics);
}