> On Dec. 17, 2015, 6:50 a.m., xiaojian zhou wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/beans/stats/StatsKey.java, > > line 304 > > <https://reviews.apache.org/r/41492/diff/1/?file=1168164#file1168164line304> > > > > You removed this stats "eventQueueSize", vsd cannot show it any more. > > > > How about fix the size in stats too?
I didn't actually remove the stat, this is just an unused constant in the management code. But you're right, the stat in vsd is still going to be wrong after this change. I do think we should probably fix it, but that fix is going to be much more difficult. Maybe I should file a separate bug for fixing that? I think the fix is to improve our stats layer so that it can poll the region size. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41492/#review110915 ----------------------------------------------------------- On Dec. 17, 2015, 2:43 a.m., Dan Smith wrote: > > ----------------------------------------------------------- > 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. > > > 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 > >
