> On Dec. 17, 2015, 5:55 a.m., Ashvin A wrote: > > Looks good. I am wondering if a unit tests for the queueSize method on > > AbstractGatewaySender could be added.
Good point. I'll see if I can add one. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41492/#review110907 ----------------------------------------------------------- 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 > >
