-----------------------------------------------------------
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.


Changes
-------

Thank you for the additional review, Sergio. I uploaded the changes accordingly.


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 (updated)
-----

  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/

Changes: https://reviews.apache.org/r/64949/diff/2-3/


Testing
-------


Thanks,

Mano Kovacs

Reply via email to