-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51557/#review147558
-----------------------------------------------------------




ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java
 (line 269)
<https://reviews.apache.org/r/51557/#comment214751>

    Can we add a unit test?


- Sumit Mohanty


On Aug. 31, 2016, 12:27 p.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51557/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2016, 12:27 p.m.)
> 
> 
> Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, 
> and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-18287
>     https://issues.apache.org/jira/browse/AMBARI-18287
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Ambari should not send env files corresponding to the deleted services for 
> the status commands.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java
>  c353049 
> 
> Diff: https://reviews.apache.org/r/51557/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>

Reply via email to