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

2017-12-22 Thread Dmitro Lisnichenko
> On Dec. 22, 2017, 5:13 p.m., Nate Cole wrote: > > What's the status of this? I thought we fixed all the outstanding Livy > > issues? It has been postponed to 2.6.2. Will open an another review request - Dmitro --- This is an

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

2017-12-22 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64667/#review194438 --- What's the status of this? I thought we fixed all the

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

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

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

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

2017-12-17 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

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

2017-12-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64667/#review194033 --- Fix it, then Ship it!

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

2017-12-17 Thread Dmitro Lisnichenko
> On Dec. 16, 2017, 10:35 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 160-164 (patched) > > > > > > We should not hard code this into

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

2017-12-16 Thread Jonathan Hurley
> On Dec. 16, 2017, 3:35 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 160-164 (patched) > > > > > > We should not hard code this into

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

2017-12-16 Thread Jonathan Hurley
> On Dec. 16, 2017, 3:35 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 160-164 (patched) > > > > > > We should not hard code this into

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

2017-12-16 Thread Dmitro Lisnichenko
> On Dec. 16, 2017, 10:35 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 160-164 (patched) > > > > > > We should not hard code this into

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

2017-12-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64667/#review194019 ---