----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41492/ -----------------------------------------------------------
(Updated Dec. 17, 2015, 2:43 a.m.) Review request for geode, Ashvin A and xiaojian zhou. Summary (updated) ----------------- GEODE-681: Measuring event queue size without using the stat for gfsh Repository: geode Description ------- The eventQueueSize stat has issues with concurrent events during start and stop that are leading it to be inaccurate. To make gfsh and jmx correct, measuring the queue size directly from the underlying region size. Diffs ----- gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/wan/AbstractGatewaySender.java 422c150fbf1e34785f79bbad54b52cd61594e64c gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/wan/AbstractGatewaySenderEventProcessor.java b9d877e1a0fab318ae8c29e8ea076bf4d42fb051 gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/wan/parallel/ParallelGatewaySenderQueue.java 94dc9e8708df08221d26791dd53ab0070657afae gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/wan/serial/ConcurrentSerialGatewaySenderEventProcessor.java 16410cec806618eff49a3a4611e257cbfa8278a6 gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/beans/GatewaySenderMBeanBridge.java 0b6f4325998b56ba03ede0dc01350f941c7e606b gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/beans/stats/StatsKey.java 50972923b5ff4842822ba30d039bb5c56f66509f Diff: https://reviews.apache.org/r/41492/diff/ Testing ------- Thanks, Dan Smith
