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


Ship it!




Ship It!

- Sergio Pena


On Jan. 10, 2018, 11:37 p.m., Mano Kovacs wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64949/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2018, 11:37 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-641
>     https://issues.apache.org/jira/browse/SENTRY-641
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Binding for Lily Hbase Indexer, based on the implementation Gregory Chanan 
> created for CDH version of Sentry. The patch contains implementation that has 
> no dependency to Lily.
> 
> 
> Diffs
> -----
> 
>   pom.xml c797181c8f123bc914ece27c13b70540978091bc 
>   sentry-binding/pom.xml b12683c1c742c989ac118e968c71df50366a8557 
>   
> sentry-binding/sentry-binding-hbase-indexer/src/main/java/org/apache/sentry/binding/hbaseindexer/authz/HBaseIndexerAuthzBinding.java
>  PRE-CREATION 
>   
> sentry-binding/sentry-binding-hbase-indexer/src/main/java/org/apache/sentry/binding/hbaseindexer/conf/HBaseIndexerAuthzConf.java
>  PRE-CREATION 
>   
> sentry-binding/sentry-binding-hbase-indexer/src/test/java/org/apache/sentry/binding/hbaseindexer/TestHBaseIndexerAuthzBinding.java
>  PRE-CREATION 
>   
> sentry-binding/sentry-binding-hbase-indexer/src/test/resources/sentry-site-service.xml
>  PRE-CREATION 
>   
> sentry-binding/sentry-binding-hbase-indexer/src/test/resources/sentry-site.xml
>  PRE-CREATION 
>   sentry-dist/pom.xml 63eb81c87172dc3237039b6d7641881b2831079f 
>   sentry-dist/src/license/THIRD-PARTY.properties 
> 22429fc3d644ea961d59260b4a08eb380d7a4325 
>   sentry-policy/pom.xml b082aa66b4324c0195ea4e7c6bdd794dbcbc18d3 
>   sentry-policy/sentry-policy-indexer/pom.xml 
> 0696b1dec5313b91d7423f9ebf4685029f2e6db1 
>   
> sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/IndexerWildcardPrivilege.java
>  9e9a0d2b3afc22098980b5a54b6a9989ad035b6f 
>   
> sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/SimpleIndexerPolicyEngine.java
>  932690c6ddad18e3c3b02b513fb80dec02a12d93 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/AbstractTestIndexerPolicyEngine.java
>  66455e8665786b48d3abf8f6659ccd22c992f130 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/IndexPolicyTestUtil.java
>  45f100e655be17748c1955f8b9925a7cbd18a81a 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestCommonPrivilegeForIndexer.java
>  aa8aa5f96b1f621a8a77ef957d5469a1feb0f583 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerAuthorizationProviderGeneralCases.java
>  29b377e9d0f560d8f38cd30ab12e81f75ea6233e 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerAuthorizationProviderSpecialCases.java
>  1717c42f831d116e3389b3a827f946128b2fd74f 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerModelAuthorizables.java
>  1d8ca539083b5bae6547aab1d7350cdb4f8ef0cd 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyEngineDFS.java
>  d166f0f981ff231cf4b323d137232fba2df677ab 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyEngineLocalFS.java
>  19ddefbb2b4af738ff9626c56a8c264b6a45c752 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyNegative.java
>  e1a0dcc55bc48adb5ad527f92d68ef6e8ace1cb6 
>   
> sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerRequiredInRole.java
>  ff20d0388df9cdbb4885ac1e74c9f8fd53ee0876 
>   sentry-policy/sentry-policy-indexer/src/test/resources/log4j.properties  
>   
> sentry-policy/sentry-policy-indexer/src/test/resources/test-authz-provider.ini
>  0e4051157c31996da251c368af2dbafe5ed665b5 
>   
> sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/AuthorizationComponent.java
>  5dc2b5592bdfbb281b083bdc835d455d07df0d86 
> 
> 
> Diff: https://reviews.apache.org/r/64949/diff/3/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mano Kovacs
> 
>

Reply via email to