----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62805/ -----------------------------------------------------------
(Updated Oct. 6, 2017, 10:39 p.m.) Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Sid Wagle. Changes ------- make sure executor mock is verified Bugs: AMBARI-22155 https://issues.apache.org/jira/browse/AMBARI-22155 Repository: ambari Description ------- Mock the executor to eliminate nondeterminism caused by threads. Diffs (updated) ----- ambari-metrics/ambari-metrics-flume-sink/src/main/java/org/apache/hadoop/metrics2/sink/flume/FlumeTimelineMetricsSink.java 62779070419894064cb39ea5cc5da012572ff4cb 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/2/ Changes: https://reviews.apache.org/r/62805/diff/1-2/ 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