----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30960/#review72935 -----------------------------------------------------------
Ship it! Ship It! - Dmytro Sen 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 > >
