----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45198/ -----------------------------------------------------------
(Updated March 25, 2016, 1:30 a.m.) Review request for sentry. Repository: sentry Description ------- With more and more components relying on sentry for authorization, it really makes sense to have the case-sensitivity configurable. We can keep the current behavior as default. However, for use cases like Kafka, it makes sense to have authorizable names case sensitive. For instance, a topic resource in Kafka can have case sensitive values, like, topic1 and TOPIC1 are two completely different valid topic resource instances. Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/policy/hive/DBPolicyTestUtil.java c390b66 sentry-binding/sentry-binding-solr/src/test/java/org/apache/sentry/policy/solr/SearchPolicyTestUtil.java 3856825 sentry-binding/sentry-binding-sqoop/src/test/java/org/apache/sentry/policy/sqoop/SqoopPolicyTestUtil.java a76554e sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ImplyMethodType.java 16a7a46 sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/KeyValue.java 079629e sentry-policy/sentry-policy-common/src/main/java/org/apache/sentry/policy/common/CommonPrivilege.java 43cb5c8 sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/ModelForTest.java a213987 sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/TestCommonPrivilege.java abaf61f sentry-policy/sentry-policy-indexer/pom.xml 1a50581 sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/IndexPolicyTestUtil.java 45d3400 Diff: https://reviews.apache.org/r/45198/diff/ Testing ------- Thanks, Colin Ma
