----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50882/ -----------------------------------------------------------
Review request for sentry. Repository: sentry Description ------- Currently, both AuthorizationProvider and ProviderBackend are in sentry-provider-common. It'll cause the circle dependency problem. Create a new module(eg, sentry-authorization-provider), and move AuthorizationProvider to this new module. Diffs ----- pom.xml 294ddb9 sentry-binding/sentry-binding-hive-common/pom.xml 685df0c sentry-binding/sentry-binding-kafka/pom.xml f868786 sentry-binding/sentry-binding-solr/pom.xml cc99948 sentry-binding/sentry-binding-sqoop/pom.xml e96802f sentry-dist/pom.xml 04645ad sentry-policy/sentry-policy-common/pom.xml 57fc9d9 sentry-policy/sentry-policy-engine/pom.xml e9c44d7 sentry-policy/sentry-policy-indexer/pom.xml e6ef72f sentry-provider/pom.xml a929b00 sentry-provider/sentry-authorization-provider/pom.xml PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/AuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/HadoopGroupResourceAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/NoAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/file/HadoopGroupResourceAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/file/LocalGroupResourceAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/test/java/org/apache/sentry/provider/common/TestGetGroupMapping.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/test/java/org/apache/sentry/provider/common/TestNoAuthorizationProvider.java PRE-CREATION sentry-provider/sentry-authorization-provider/src/test/java/org/apache/sentry/provider/file/TestLocalGroupMapping.java PRE-CREATION sentry-provider/sentry-provider-cache/pom.xml 694df36 sentry-provider/sentry-provider-common/pom.xml f83f594 sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/AuthorizationProvider.java 3d6440f sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/HadoopGroupResourceAuthorizationProvider.java 6e5dbc3 sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/NoAuthorizationProvider.java 11dbfb7 sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java a6b2047 sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/file/HadoopGroupResourceAuthorizationProvider.java bf2c5a1 sentry-provider/sentry-provider-common/src/test/java/org/apache/sentry/provider/common/TestGetGroupMapping.java f6d8c05 sentry-provider/sentry-provider-common/src/test/java/org/apache/sentry/provider/common/TestNoAuthorizationProvider.java 7ca8bfc sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupMappingService.java 7e570ae sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/LocalGroupResourceAuthorizationProvider.java a9e7836 sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/SimpleFileProviderBackend.java 69ab260 sentry-provider/sentry-provider-file/src/test/java/org/apache/sentry/provider/file/TestLocalGroupMapping.java 9864b82 Diff: https://reviews.apache.org/r/50882/diff/ Testing ------- Thanks, Colin Ma
