> On Фев. 16, 2015, 3:35 п.п., Dmytro Sen wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/metrics.json, > > line 137 > > <https://reviews.apache.org/r/30960/diff/1/?file=862735#file862735line137> > > > > I think, > > > > "metrics/kafka/log/LogFlushStats/LogFlushRateAndTimeMs/max" should look > > like "metrics/kafka/log/LogFlushStats/LogFlushRateAndTimeMs/rate/max" > > and > > "kafka.log.LogFlushStats.LogFlushRateAndTimeMs.max" should look like > > "kafka.log.LogFlushStats.LogFlushRateAndTimeMs.max"_rate._max" > > > > Aggregation functions has been intoroduced in AMBARI-9604 Fix Flume > > Agent graphs on Ambari service page > > > > But I can be wrong if kafka sends min/max/avg metrics itself > > Florian Barca wrote: > Thanks Dmitry, I will check, so far I positively tested the syntax, with > direct HTTP requests as well as from the fully-deployed product, and the > graph shows up as it's supposed to.
Kafka generates the stats directly - Dmytro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30960/#review72618 ----------------------------------------------------------- On Фев. 17, 2015, 4:32 п.п., Florian Barca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30960/ > ----------------------------------------------------------- > > (Updated Фев. 17, 2015, 4:32 п.п.) > > > Review request for Ambari, Dmytro Sen, Erik Bergenholtz, Srimanth Gunturi, > and Yusaku Sako. > > > Bugs: AMBARI-9612 > https://issues.apache.org/jira/browse/AMBARI-9612 > > > Repository: ambari > > > Description > ------- > > Fixed log flush status collection & display. Feeding the right data to the > chart. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/metrics.json > 826c0e8a35fd90a425ed2bd029e4385040e8f41f > > ambari-server/src/main/resources/stacks/HDP/2.2.GlusterFS/services/KAFKA/metrics.json > 826c0e8a35fd90a425ed2bd029e4385040e8f41f > ambari-web/app/messages.js f794271fd0e4a6205a14654a46e61197d69a9bda > ambari-web/app/utils/ajax/ajax.js bfbc2dd570be2b67385bda31f2789b1345ae7b1a > ambari-web/app/views/main/service/info/metrics/kafka/kafka_log_flush.js > 240a89b854a147ea31ae1b83248d7c5cd08a8d7d > > Diff: https://reviews.apache.org/r/30960/diff/ > > > Testing > ------- > > mvn clean test > deployment on a local VMs cluster -> generated Kafka messages -> log flush > status graph gets populated ok > > No unit tests needed, there are tests available already for the chart class > ChartLinearTimeView (see > ambari-web/test/views/common/chart/linear_time_test.js), and the view uses > this class directly, without modifying its behavior. > > > Thanks, > > Florian Barca > >