----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45198/ -----------------------------------------------------------
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 ----- 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 Diff: https://reviews.apache.org/r/45198/diff/ Testing ------- Thanks, Colin Ma
