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