You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Colm O hEigeartaigh <co...@apache.org> on 2017/12/05 11:05:45 UTC

Re: Review Request 64051: RANGER-1906 - Simplify Atlas plugin dependency management


> On Nov. 30, 2017, 6:52 a.m., Mehul Parikh wrote:
> > @Colm : Can you please confirm if Audit to HDFS and Audit to Solr are working after removal of these dependencies?
> 
> Colm O hEigeartaigh wrote:
>     Hi Mehul,
>     
>     I haven't actually removed any dependencies with this patch as such - the distribution jars are exactly the same. The Solr jar is bundled via the agents-common dependency, and the Atlas plugin takes the Hadoop jars from Atlas itself instead of via the lib directory in the distribution.

@Mehul, As a sanity test I've verified that Atlas auditing to Solr works as expected.


- Colm


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64051/#review192242
-----------------------------------------------------------


On Nov. 28, 2017, 11:44 a.m., Colm O hEigeartaigh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64051/
> -----------------------------------------------------------
> 
> (Updated Nov. 28, 2017, 11:44 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-1906
>     https://issues.apache.org/jira/browse/RANGER-1906
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> This task is to simplify the dependency management for the Atlas plugin. In particular, the hadoop/solr dependencies should just be imported via the ranger-plugin-commons dependency.
> 
> No changes are made to the resulting jars for the Atlas distribution.
> 
> 
> Diffs
> -----
> 
>   plugin-atlas/pom.xml 957b4ce3 
>   ranger-atlas-plugin-shim/pom.xml a207d16b 
>   src/main/assembly/plugin-atlas.xml fd988116 
> 
> 
> Diff: https://reviews.apache.org/r/64051/diff/2/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>