----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62805/#review187278 -----------------------------------------------------------
Ship it! ambari-metrics/ambari-metrics-flume-sink/src/test/java/org/apache/hadoop/metrics2/sink/flume/FlumeTimelineMetricsSinkTest.java Lines 84 (patched) <https://reviews.apache.org/r/62805/#comment264197> Missing expectLastCall() ? - Sid Wagle On Oct. 6, 2017, 9:47 a.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62805/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2017, 9:47 a.m.) > > > Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle. > > > Bugs: AMBARI-22155 > https://issues.apache.org/jira/browse/AMBARI-22155 > > > Repository: ambari > > > Description > ------- > > Mock the executor to eliminate nondeterminism caused by threads. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-flume-sink/src/test/java/org/apache/hadoop/metrics2/sink/flume/FlumeTimelineMetricsSinkTest.java > bd4ae6af88ee13f4ba6a3f14ac8c138b72d63c9d > > > Diff: https://reviews.apache.org/r/62805/diff/1/ > > > Testing > ------- > > ``` > $ mvn -am -pl ambari-metrics/ambari-metrics-flume-sink clean test > ... > Tests run: 5, Failures: 0, Errors: 0, Skipped: 0 > ... > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari Main ........................................ SUCCESS [ 6.159 > s] > [INFO] Apache Ambari Project POM .......................... SUCCESS [ 0.036 > s] > [INFO] ambari-utility ..................................... SUCCESS [ 3.880 > s] > [INFO] ambari-metrics ..................................... SUCCESS [ 0.594 > s] > [INFO] Ambari Metrics Common .............................. SUCCESS [ 4.240 > s] > [INFO] Ambari Metrics Flume Sink .......................... SUCCESS [ 4.697 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > ``` > > > Thanks, > > Attila Doroszlai > >
