Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-23 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63981/#review191791 --- Ship it! Ship It! - Colm O hEigeartaigh On Nov. 23, 2017, 3:

Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-22 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63981/ --- (Updated 十一月 23, 2017, 3:19 a.m.) Review request for ranger, Alok Lal, Ankita S

Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-22 Thread pengjianhua
> On 十一月 22, 2017, 中午, Colm O hEigeartaigh wrote: > > You could also remove the solr dependency - it is picked up anyway from the > > plugins-audit dependency, and shouldn't be listed explicitly in the hdfs > > pom, as there are no solr references in that module. Ok,I have tested solr really n

Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-22 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63981/#review191718 --- You could also remove the solr dependency - it is picked up anyway

Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-22 Thread Qiang Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63981/#review191715 --- Ship it! Ship It! - Qiang Zhang On Nov. 22, 2017, 2:04 a.m.,

Re: Review Request 63981: RANGER-1903:Simplify the maven dependency management of the Hdfs plugin for Ranger.

2017-11-21 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63981/ --- (Updated 十一月 22, 2017, 2:04 a.m.) Review request for ranger, Alok Lal, Ankita S