dschneider-pivotal commented on a change in pull request #7358:
URL: https://github.com/apache/geode/pull/7358#discussion_r808339991
##########
File path:
geode-for-redis/src/main/java/org/apache/geode/redis/internal/statistics/RedisStats.java
##########
@@ -39,15 +39,14 @@
private final AtomicLong uniqueChannelSubscriptions = new AtomicLong();
private final AtomicLong uniquePatternSubscriptions = new AtomicLong();
- private final int rollingAverageSamplesPerSecond = 16;
+ private static final int ROLLING_AVERAGE_SAMPLES_PER_SECOND = 16;
private final ScheduledExecutorService rollingAverageExecutor;
- private volatile double networkKiloBytesReadOverLastSecond;
- private final long[] networkBytesReadOverLastNSamples = new
long[rollingAverageSamplesPerSecond];
- private long totalNetworkBytesReadLastTick;
- private double opsPerformedOverLastSecond;
- private volatile long totalOpsPerformedLastTick;
- private final long[] opsPerformedOverLastNSamples = new
long[rollingAverageSamplesPerSecond];
private int rollingAverageTick = 0;
+ private final RollingUpgradeStat networkKiloBytesReadRollingAverageStat =
Review comment:
should this be named "Bytes" instead of "KiloBytes" to make clear that
the units it computes is bytes and the caller needs to convert then to
kilobytes by dividing by 1024?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
