Review Request 64693: Add service group dependencies

2017-12-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64693/ --- Review request for Ambari, Jayush Luniya, Madhuvanthi Radhakrishnan, and Swapan

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-18 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review194085 --- Ship it! Ship It! - Nate Cole On Dec. 13, 2017, 5:01 p.m., R

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-18 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review194074 --- Ship it! Ship It! - Eugene Chekanskiy On Dec. 13, 2017, 10:0

Re: Review Request 64667: Livy server fails to start during downgrade due to absence of 'conf' directory

2017-12-18 Thread Jonathan Hurley
> On Dec. 17, 2017, 2:14 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 155 (patched) > > > > > > OK - let's just make this a little less hard-c

Re: Review Request 64667: Livy server fails to start during downgrade due to absence of 'conf' directory

2017-12-18 Thread Dmitro Lisnichenko
> On Dec. 17, 2017, 9:14 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 155 (patched) > > > > > > OK - let's just make this a little less hard-c

Re: Review Request 64667: Livy server fails to start during downgrade due to absence of 'conf' directory

2017-12-18 Thread Dmitro Lisnichenko
> On Dec. 17, 2017, 9:14 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 155 (patched) > > > > > > OK - let's just make this a little less hard-c

Re: Review Request 64682: Fix unit test failures caused by old json format

2017-12-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64682/#review194067 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 18, 2017, 1:3

Review Request 64682: Fix unit test failures caused by old json format

2017-12-18 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64682/ --- Review request for Ambari, Attila Doroszlai, Myroslav Papirkovskyy, Sid Wagle, a

Re: Review Request 64679: Log Search UI: incorrect caption for graph gap in weeks

2017-12-18 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64679/#review194063 --- Ship it! Ship It! - Aleksandr Kovalenko On Дек. 18, 2017, 12

Review Request 64679: Log Search UI: incorrect caption for graph gap in weeks

2017-12-18 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64679/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-22663 https

Re: Review Request 64670: AMBARI-22660. Fix unit tests failing due to ServiceGroup issues

2017-12-18 Thread Attila Doroszlai
> On Dec. 18, 2017, noon, Balázs Bence Sári wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > > Line 625 (original), 629 (patched) > > > > > > Wonder if

Re: Review Request 64670: AMBARI-22660. Fix unit tests failing due to ServiceGroup issues

2017-12-18 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64670/#review194057 --- Ship it! Ship It! - Balázs Bence Sári On Dec. 17, 2017, 1:35

Re: Review Request 64670: AMBARI-22660. Fix unit tests failing due to ServiceGroup issues

2017-12-18 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64670/#review194055 --- ambari-server/src/main/java/org/apache/ambari/server/controller/A

Re: Review Request 64660: Rafactor: Package Log Search and Log Feeder into jars + cleanup start scripts

2017-12-18 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64660/#review194044 --- Ship it! Ship It! - Attila Magyar On Dec. 16, 2017, 1:27 p.m