----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34418/#review84344 -----------------------------------------------------------
Looks good. minor comment clients/src/main/java/org/apache/kafka/common/metrics/stats/Rate.java <https://reviews.apache.org/r/34418/#comment135563> It's still possible to configure the samples() in MetricConfig to 1. Should we change the assert to ensure we have at least 2 samples always. clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java <https://reviews.apache.org/r/34418/#comment135560> can you remove this? - Aditya Auradkar On May 19, 2015, 5:12 p.m., Jay Kreps wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34418/ > ----------------------------------------------------------- > > (Updated May 19, 2015, 5:12 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-2191 > https://issues.apache.org/jira/browse/KAFKA-2191 > > > Repository: kafka > > > Description > ------- > > Fix 0 ellapsed time rate bug. > > > Diffs > ----- > > clients/src/main/java/org/apache/kafka/common/metrics/stats/Rate.java > 98429da34418f7f1deba1b5e44e2e6025212edb3 > clients/src/test/java/org/apache/kafka/common/metrics/MetricsTest.java > 544e120594de78c43581a980b1e4087b4fb98ccb > > Diff: https://reviews.apache.org/r/34418/diff/ > > > Testing > ------- > > > Thanks, > > Jay Kreps > >
