Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/#review147567 --- Ship it! Ship It! - Sumit Mohanty On Sept. 1, 2016, 2:59 p.m

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Vitalyi Brodetskyi
> On Вер. 1, 2016, 2:13 після полудня, Sumit Mohanty wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java, > > line 269 > > > > > > Can we add a unit test? Done - Vitalyi

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Vitalyi Brodetskyi
--- 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 On

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-09-01 Thread Sumit Mohanty
--- 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/Heartb

Re: Review Request 51557: status commands get all the *-env files irrespective of a service exists or not

2016-08-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51557/#review147424 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 31, 2016, 3:2