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 2016/04/25 11:49:05 UTC

Review Request 46628: Cleanup test dependencies

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

Review request for ranger.


Repository: ranger


Description
-------

Some tasks to cleanup test dependencies:

a) Remove hamcrest + mockito from modules where they are not needed
b) Remove unused powermock dependency from root pom
c) Remove hamcrest from assembly files


Diffs
-----

  agents-cred/pom.xml 673dbc1 
  credentialbuilder/pom.xml bf34222 
  hbase-agent/pom.xml 84adc75 
  hdfs-agent/pom.xml 5918f9f 
  kms/pom.xml 03f7992 
  knox-agent/pom.xml 77e27fb 
  pom.xml d331397 
  ranger-hbase-plugin-shim/pom.xml 0b68216 
  ranger-hdfs-plugin-shim/pom.xml 0fa9e7c 
  ranger-hive-plugin-shim/pom.xml 4fabb3f 
  ranger-knox-plugin-shim/pom.xml 8a97917 
  security-admin/pom.xml e6c8edf 
  src/main/assembly/admin-web.xml 8087374 
  src/main/assembly/hbase-agent.xml 9169248 
  src/main/assembly/hdfs-agent.xml 978b1f6 
  src/main/assembly/hive-agent.xml 611b71b 
  src/main/assembly/kms.xml 445b3c7 
  src/main/assembly/knox-agent.xml 431af88 
  src/main/assembly/plugin-kafka.xml a860a7b 
  src/main/assembly/plugin-kms.xml 749894d 
  src/main/assembly/plugin-solr.xml 29e32f9 
  src/main/assembly/plugin-yarn.xml 200694f 
  src/main/assembly/storm-agent.xml be88dd7 
  ugsync/pom.xml 877ac7f 

Diff: https://reviews.apache.org/r/46628/diff/


Testing
-------


Thanks,

Colm O hEigeartaigh


Re: Review Request 46628: RANGER-948 - Cleanup test dependencies

Posted by Colm O hEigeartaigh <co...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46628/
-----------------------------------------------------------

(Updated April 26, 2016, 9:43 a.m.)


Review request for ranger.


Summary (updated)
-----------------

RANGER-948 - Cleanup test dependencies


Repository: ranger


Description
-------

Some tasks to cleanup test dependencies:

a) Remove hamcrest + mockito from modules where they are not needed
b) Remove unused powermock dependency from root pom
c) Remove hamcrest from assembly files


Diffs
-----

  agents-cred/pom.xml 673dbc1 
  credentialbuilder/pom.xml bf34222 
  hbase-agent/pom.xml 84adc75 
  hdfs-agent/pom.xml 5918f9f 
  kms/pom.xml 03f7992 
  knox-agent/pom.xml 77e27fb 
  pom.xml d331397 
  ranger-hbase-plugin-shim/pom.xml 0b68216 
  ranger-hdfs-plugin-shim/pom.xml 0fa9e7c 
  ranger-hive-plugin-shim/pom.xml 4fabb3f 
  ranger-knox-plugin-shim/pom.xml 8a97917 
  security-admin/pom.xml e6c8edf 
  src/main/assembly/admin-web.xml 8087374 
  src/main/assembly/hbase-agent.xml 9169248 
  src/main/assembly/hdfs-agent.xml 978b1f6 
  src/main/assembly/hive-agent.xml 611b71b 
  src/main/assembly/kms.xml 445b3c7 
  src/main/assembly/knox-agent.xml 431af88 
  src/main/assembly/plugin-kafka.xml a860a7b 
  src/main/assembly/plugin-kms.xml 749894d 
  src/main/assembly/plugin-solr.xml 29e32f9 
  src/main/assembly/plugin-yarn.xml 200694f 
  src/main/assembly/storm-agent.xml be88dd7 
  ugsync/pom.xml 877ac7f 

Diff: https://reviews.apache.org/r/46628/diff/


Testing
-------


Thanks,

Colm O hEigeartaigh


Re: Review Request 46628: Cleanup test dependencies

Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46628/#review130456
-----------------------------------------------------------


Ship it!




Colm - please include JIRA number in reviews; it helps to be able to cross reference.

- Madhan Neethiraj


On April 25, 2016, 9:49 a.m., Colm O hEigeartaigh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46628/
> -----------------------------------------------------------
> 
> (Updated April 25, 2016, 9:49 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Some tasks to cleanup test dependencies:
> 
> a) Remove hamcrest + mockito from modules where they are not needed
> b) Remove unused powermock dependency from root pom
> c) Remove hamcrest from assembly files
> 
> 
> Diffs
> -----
> 
>   agents-cred/pom.xml 673dbc1 
>   credentialbuilder/pom.xml bf34222 
>   hbase-agent/pom.xml 84adc75 
>   hdfs-agent/pom.xml 5918f9f 
>   kms/pom.xml 03f7992 
>   knox-agent/pom.xml 77e27fb 
>   pom.xml d331397 
>   ranger-hbase-plugin-shim/pom.xml 0b68216 
>   ranger-hdfs-plugin-shim/pom.xml 0fa9e7c 
>   ranger-hive-plugin-shim/pom.xml 4fabb3f 
>   ranger-knox-plugin-shim/pom.xml 8a97917 
>   security-admin/pom.xml e6c8edf 
>   src/main/assembly/admin-web.xml 8087374 
>   src/main/assembly/hbase-agent.xml 9169248 
>   src/main/assembly/hdfs-agent.xml 978b1f6 
>   src/main/assembly/hive-agent.xml 611b71b 
>   src/main/assembly/kms.xml 445b3c7 
>   src/main/assembly/knox-agent.xml 431af88 
>   src/main/assembly/plugin-kafka.xml a860a7b 
>   src/main/assembly/plugin-kms.xml 749894d 
>   src/main/assembly/plugin-solr.xml 29e32f9 
>   src/main/assembly/plugin-yarn.xml 200694f 
>   src/main/assembly/storm-agent.xml be88dd7 
>   ugsync/pom.xml 877ac7f 
> 
> Diff: https://reviews.apache.org/r/46628/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Colm O hEigeartaigh
> 
>