You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Hao Hao <ha...@cloudera.com> on 2016/08/02 20:36:13 UTC

Re: Review Request 50474: SENTRY-1404: Use the new INodeAttributesProvider API in sentry-hdfs

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

(Updated Aug. 2, 2016, 8:36 p.m.)


Review request for sentry and Sravya Tirukkovalur.


Repository: sentry


Description
-------

Use the new INodeAttributesProvider API in sentry-hdfs


Diffs (updated)
-----

  pom.xml a4f2bcc14855a2e5f712f9c56fbdb1e137edfa93 
  sentry-binding/sentry-binding-solr/pom.xml 8b94c87118841c2ce775c3e79ff6bb43ba5185fd 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/hadoop/hdfs/server/namenode/AuthorizationProvider.java 383d64d88c74c93cd79ccb189301b9c1a6d919e4 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationConstants.java 883680182fd01f4b0b1a7fb5d8eb4d7f23f5851e 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationProvider.java f639f5fbacc85b2adb757087c2eba66b092743a2 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryINodeAttributesProvider.java PRE-CREATION 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/MockSentryAuthorizationProvider.java 2085b525db0b7a9fda77a4be62026da3bf74b7ac 
  sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/TestSentryAuthorizationProvider.java 5da0dc2fb4a47098bd5f9b36dbacb0296162f95b 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java 60085b20774fb28c978f60b97bbef0ecbadb8d8f 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java 25d13d62a1a16beba16ae46fbd28f2490fca10bc 

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


Testing
-------


Thanks,

Hao Hao


Re: Review Request 50474: SENTRY-1404: Use the new INodeAttributesProvider API in sentry-hdfs

Posted by Anne Yu <an...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50474/#review144810
-----------------------------------------------------------


Ship it!




Ship It!

- Anne Yu


On Aug. 2, 2016, 8:36 p.m., Hao Hao wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50474/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2016, 8:36 p.m.)
> 
> 
> Review request for sentry and Sravya Tirukkovalur.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Use the new INodeAttributesProvider API in sentry-hdfs
> 
> 
> Diffs
> -----
> 
>   pom.xml a4f2bcc14855a2e5f712f9c56fbdb1e137edfa93 
>   sentry-binding/sentry-binding-solr/pom.xml 8b94c87118841c2ce775c3e79ff6bb43ba5185fd 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/hadoop/hdfs/server/namenode/AuthorizationProvider.java 383d64d88c74c93cd79ccb189301b9c1a6d919e4 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationConstants.java 883680182fd01f4b0b1a7fb5d8eb4d7f23f5851e 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationProvider.java f639f5fbacc85b2adb757087c2eba66b092743a2 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryINodeAttributesProvider.java PRE-CREATION 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/MockSentryAuthorizationProvider.java 2085b525db0b7a9fda77a4be62026da3bf74b7ac 
>   sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/TestSentryAuthorizationProvider.java 5da0dc2fb4a47098bd5f9b36dbacb0296162f95b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java 60085b20774fb28c978f60b97bbef0ecbadb8d8f 
>   sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java 25d13d62a1a16beba16ae46fbd28f2490fca10bc 
> 
> Diff: https://reviews.apache.org/r/50474/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hao Hao
> 
>