Re: Review Request 49260: Update desired states in case of service restarts

2016-06-29 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139973 --- Ship it! Ship It! - Sandor Magyari On June 29, 2016, 9:10

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-29 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/ --- (Updated June 29, 2016, 9:10 a.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-29 Thread Laszlo Puskas
> On June 29, 2016, 1:20 a.m., Sumit Mohanty wrote: > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java, > > line 20 > > > > > > Are changes to this file

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-28 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139911 --- Ship it! Verify if changes to

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-28 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139909 ---

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-28 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/ --- (Updated June 28, 2016, 2:09 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-28 Thread Laszlo Puskas
> On June 27, 2016, 2:32 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java, > > line 482 > > > > > > Lets modify one of the

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139577 ---

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139567 --- Ship it! Ship It! - Oliver Szabo On June 27, 2016, 12:42

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139565 --- Ship it! Ship It! - Sandor Magyari On June 27, 2016, 12:42

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/#review139563 --- Ship it! Ship It! - Daniel Gergely On jún. 27, 2016, 12:42

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/ --- (Updated June 27, 2016, 12:42 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/ --- (Updated June 27, 2016, 12:12 p.m.) Review request for Ambari, Daniel Gergely,

Review Request 49260: Update desired states in case of service restarts

2016-06-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49260/ --- Review request for Ambari, Daniel Gergely, Oliver Szabo, Robert Nettleton, and