Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/#review144140 --- Ship it! Ship It! - Alejandro Fernandez On July 29, 2016, 12

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-29 Thread Mugdha Varadkar
> On July 28, 2016, 4:16 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_usersync.py, > > line 74 > > > > > > I don't think you need the `upgrade_

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-29 Thread Mugdha Varadkar
> On July 28, 2016, 8:10 p.m., Nate Cole wrote: > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/config-upgrade.xml, > > lines 659-665 > > > > > > I think you might be able to use the "modern" here: > >

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-29 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/ --- (Updated July 29, 2016, 12:16 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/#review143990 --- ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/config-u

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/#review143991 --- Ship it! Concur with Jonathan's comments, then Ship It - Nate

Re: Review Request 50565: AMBARI-17945 Ranger UserSync restart failed during EU from 2.2.9 to 2.4.2

2016-07-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/#review143950 --- Fix it, then Ship it! ambari-server/src/main/resources/common-