-----------------------------------------------------------
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

Reply via email to