Re: Review Request 59339: RANGER-1587 - Cleanup Ranger KMS service dependencies

2017-05-25 Thread Qiang Zhang

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


Ship it!




Ship It!

- Qiang Zhang


On 五月 17, 2017, 3:06 p.m., Colm O hEigeartaigh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59339/
> ---
> 
> (Updated 五月 17, 2017, 3:06 p.m.)
> 
> 
> Review request for ranger.
> 
> 
> Bugs: RANGER-1587
> https://issues.apache.org/jira/browse/RANGER-1587
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> The Ranger KMS service pom is a bit of a mess, with a large number of 
> dependencies listed - most of which can be imported via Hadoop. I've made the 
> following changes (and verified that the jars in the KMS distribution are the 
> exact same as before):
> 
> Removed:
> commons-digester 1.8 (comes from commons-configuration 1.6)
> commons-configuration 1.6 (comes from Hadoop 2.7.1)
> javax.activation 1.1 (comes from jaxb-api 2.2.2)
> jaxb-api 2.2.2 (comes from jaxb-impl 2.2.3-1)
> eclipselink + javax.persistence (comes from ranger-plugins-audit)
> commons-beanutils 1.7.0 (comes from commons-digester 1.8)
> commons-beanutils-core 1.8.0 (comes from commons-configuration 1.6)
> commons-compress 1.4.1 (comes from hadoop 2.7.1)
> commons-math3 3.1.1 (comes from hadoop 2.7.1)
> commons-net 3.1 (comes from hadoop 2.7.1)
> apacheds-i18n 2.0.0-M15 (comes from apacheds-kerberos-codec 2.0.0-M15)
> api-asn1-api 1.0.0-M20 (comes from apacheds-kerberos-codec 2.0.0-M15)
> api-util 1.0.0-M20 (comes from apacheds-kerberos-codec 2.0.0-M15)
> apacheds-i18n 2.0.0-M15 (comes from apacheds-kerberos-codec 2.0.0-M15)
> apacheds-kerberos-codec 2.0.0-M15 (comes from hadoop-auth 2.7.1)
> avro 1.7.4 (comes from hadoop-common 2.7.1)
> commons-cli 1.2 (comes from hadoop-common 2.7.1)
> commons-io 2.4 (comes from hadoop-common 2.7.1)
> commons-codec 1.4 (comes from hadoop-common 2.7.1)
> gson 2.2.4 (comes from hadoop-common 2.7.1)
> hadoop-annotations 2.7.1 (comes from hadoop-common 2.7.1)
> htrace 3.1.0-incubating (comes from hadoop-common 2.7.1)
> jaxb-impl:jar:2.2.3-1 (comes from jersey-json 1.19.3)
> jettison 1.1 (comes from jersey-json 1.19.3)
> jsch 0.1.42 (comes from hadoop-common 2.7.1)
> paranamer 2.3 (comes from avro 1.7.4)
> protobuf-java 2.5.0 (comes from hadoop-common 2.7.1)
> snappy-java 1.0.4.1 (comes from avro 1.7.4)
> xmlenc 0.52 (comes from hadoop-common 2.7.1)
> xz 1.0 (comes from commons-compress 1.4.1)
> noggit 0.6 (comes from solrj 5.5.1)
> 
> 
> Diffs
> -
> 
>   agents-audit/pom.xml 10daa694 
>   kms/pom.xml 46dc3c48 
>   pom.xml 19ceca85 
> 
> 
> Diff: https://reviews.apache.org/r/59339/diff/1/
> 
> 
> Testing
> ---
> 
> Tested that the KMS dist has the same jars as before.
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>



Review Request 59339: RANGER-1587 - Cleanup Ranger KMS service dependencies

2017-05-17 Thread Colm O hEigeartaigh

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

Review request for ranger.


Bugs: RANGER-1587
https://issues.apache.org/jira/browse/RANGER-1587


Repository: ranger


Description
---

The Ranger KMS service pom is a bit of a mess, with a large number of 
dependencies listed - most of which can be imported via Hadoop. I've made the 
following changes (and verified that the jars in the KMS distribution are the 
exact same as before):

Removed:
commons-digester 1.8 (comes from commons-configuration 1.6)
commons-configuration 1.6 (comes from Hadoop 2.7.1)
javax.activation 1.1 (comes from jaxb-api 2.2.2)
jaxb-api 2.2.2 (comes from jaxb-impl 2.2.3-1)
eclipselink + javax.persistence (comes from ranger-plugins-audit)
commons-beanutils 1.7.0 (comes from commons-digester 1.8)
commons-beanutils-core 1.8.0 (comes from commons-configuration 1.6)
commons-compress 1.4.1 (comes from hadoop 2.7.1)
commons-math3 3.1.1 (comes from hadoop 2.7.1)
commons-net 3.1 (comes from hadoop 2.7.1)
apacheds-i18n 2.0.0-M15 (comes from apacheds-kerberos-codec 2.0.0-M15)
api-asn1-api 1.0.0-M20 (comes from apacheds-kerberos-codec 2.0.0-M15)
api-util 1.0.0-M20 (comes from apacheds-kerberos-codec 2.0.0-M15)
apacheds-i18n 2.0.0-M15 (comes from apacheds-kerberos-codec 2.0.0-M15)
apacheds-kerberos-codec 2.0.0-M15 (comes from hadoop-auth 2.7.1)
avro 1.7.4 (comes from hadoop-common 2.7.1)
commons-cli 1.2 (comes from hadoop-common 2.7.1)
commons-io 2.4 (comes from hadoop-common 2.7.1)
commons-codec 1.4 (comes from hadoop-common 2.7.1)
gson 2.2.4 (comes from hadoop-common 2.7.1)
hadoop-annotations 2.7.1 (comes from hadoop-common 2.7.1)
htrace 3.1.0-incubating (comes from hadoop-common 2.7.1)
jaxb-impl:jar:2.2.3-1 (comes from jersey-json 1.19.3)
jettison 1.1 (comes from jersey-json 1.19.3)
jsch 0.1.42 (comes from hadoop-common 2.7.1)
paranamer 2.3 (comes from avro 1.7.4)
protobuf-java 2.5.0 (comes from hadoop-common 2.7.1)
snappy-java 1.0.4.1 (comes from avro 1.7.4)
xmlenc 0.52 (comes from hadoop-common 2.7.1)
xz 1.0 (comes from commons-compress 1.4.1)
noggit 0.6 (comes from solrj 5.5.1)


Diffs
-

  agents-audit/pom.xml 10daa694 
  kms/pom.xml 46dc3c48 
  pom.xml 19ceca85 


Diff: https://reviews.apache.org/r/59339/diff/1/


Testing
---

Tested that the KMS dist has the same jars as before.


Thanks,

Colm O hEigeartaigh