----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48432/#review138046 -----------------------------------------------------------
geode-core/src/main/java/com/gemstone/gemfire/Statistics.java (line 469) <https://reviews.apache.org/r/48432/#comment203266> I didn't see how the impl would throw ArrayIndexOutOfBoundsException geode-core/src/main/java/com/gemstone/gemfire/Statistics.java (line 473) <https://reviews.apache.org/r/48432/#comment203262> Do we want "since" javadocs tags on new external apis? geode-core/src/main/java/com/gemstone/gemfire/Statistics.java (line 498) <https://reviews.apache.org/r/48432/#comment203267> get rid of the duplicate IllegalArgumentException geode-core/src/main/java/com/gemstone/gemfire/internal/StatSamplerStats.java (line 63) <https://reviews.apache.org/r/48432/#comment203264> On "gauge" descriptions it is better to say "Current number" and use "Total number" for counters. geode-core/src/main/java/com/gemstone/gemfire/internal/StatisticsImpl.java (line 58) <https://reviews.apache.org/r/48432/#comment203265> why not final? - Darrel Schneider On June 9, 2016, 11:41 a.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48432/ > ----------------------------------------------------------- > > (Updated June 9, 2016, 11:41 a.m.) > > > Review request for geode, anilkumar gingade, Darrel Schneider, Jason Huynh, > Jinmei Liao, Kirk Lund, and nabarun nag. > > > Repository: geode > > > Description > ------- > > Adding support to measure statistics with suppliers that are sampled > every sample interval. The suppliers are invoked in a separate thread > than the stat sampler so that slow suppliers won't screw up all stats. > > > Diffs > ----- > > geode-core/src/main/java/com/gemstone/gemfire/Statistics.java > d7f45a50af7ce58e0d65bfd6300821b36864cc9b > > geode-core/src/main/java/com/gemstone/gemfire/internal/DummyStatisticsImpl.java > 83d40559ba652474630d04758696f2e9042910e7 > geode-core/src/main/java/com/gemstone/gemfire/internal/HostStatSampler.java > ebb4177c12f6d67ef2cafb76d7a3d765fb1139ea > > geode-core/src/main/java/com/gemstone/gemfire/internal/StatSamplerStats.java > c70247939a50392e62e264bfb783a37c93a969ae > geode-core/src/main/java/com/gemstone/gemfire/internal/StatisticsImpl.java > 2344d9315ca8b2b31e9eba35a350eca44a9b8bec > > geode-core/src/main/java/com/gemstone/gemfire/internal/StatisticsTypeImpl.java > 2a29d46dfc260a40b018af5530a4f1e781e72274 > > geode-core/src/main/java/com/gemstone/gemfire/internal/statistics/CallbackSampler.java > PRE-CREATION > > geode-core/src/main/java/com/gemstone/gemfire/internal/statistics/SampleCollector.java > dfb502bb0aa018a11ab26b10a35da2c5be7c10ac > > geode-core/src/test/java/com/gemstone/gemfire/internal/LocalStatisticsImplJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/StatArchiveWriterReaderJUnitTest.java > bb15ff2ce747a37d87c95931d5e07cbce4ca7be6 > > geode-core/src/test/java/com/gemstone/gemfire/internal/StatSamplerJUnitTest.java > f2946b619c6272539ba00f10a9fb1444cc095231 > > geode-core/src/test/java/com/gemstone/gemfire/internal/statistics/CallbackSamplerJUnitTest.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/statistics/DummyStatistics.java > a1ea030443f096ba8f0bd8d5fb62a8183f20bcfd > > Diff: https://reviews.apache.org/r/48432/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >