> On Вер. 1, 2016, 2:13 після полудня, Sumit Mohanty wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java,
> >  line 269
> > <https://reviews.apache.org/r/51557/diff/1/?file=1489120#file1489120line269>
> >
> >     Can we add a unit test?

Done


- Vitalyi


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


On Вер. 1, 2016, 2:59 після полудня, Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51557/
> -----------------------------------------------------------
> 
> (Updated Вер. 1, 2016, 2:59 після полудня)
> 
> 
> 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 
>   
> ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatMonitor.java
>  a5396d8 
> 
> Diff: https://reviews.apache.org/r/51557/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>

Reply via email to