Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144865 --- Ship it! Ship It! - Gautam Borad On Aug. 4, 2016, 6:48 p.m.,

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144834 --- Ship it! Ship It! - Jayush Luniya On Aug. 4, 2016, 6:48 p.m.

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144821 --- Ship it! Ship It! - Alejandro Fernandez On Aug. 4, 2016, 6:4

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Mugdha Varadkar
> On Aug. 4, 2016, 4:39 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.7.0.2.5/kerberos.json, > > line 13 > > > > > > folder should be common-services/ATLAS/0.7.0. The "2.5" i

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Mugdha Varadkar
> On Aug. 4, 2016, 4:48 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.7.0.2.5/configuration/ranger-atlas-policymgr-ssl.xml, > > line 25 > > > > > > We should not setting any

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/ --- (Updated Aug. 4, 2016, 6:48 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Alejandro Fernandez
> On Aug. 4, 2016, 4:51 p.m., Jayush Luniya wrote: > > I would recommend that break this patch into 2 parts > > 1. Fix for the issue itself that can go into trunk and 2.4 > > 2. Refactoring into common-services that can go in trunk and 2.5 > > > > That way you are not blocked on 2.4 commit. Ag

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144782 --- I would recommend that break this patch into 2 parts 1. Fix for t

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144781 --- ambari-server/src/main/resources/common-services/ATLAS/0.7.0.2.5/

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144780 --- ambari-server/src/main/resources/common-services/ATLAS/0.7.0.2.5/

Re: Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/#review144757 --- Ship it! Ship It! - Robert Levas On Aug. 4, 2016, 8:39 a.m.,

Review Request 50791: AMBARI-18019 Referring component level identity is not working for ranger-atlas-plugin

2016-08-04 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50791/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Rob