-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43242/
-----------------------------------------------------------

(Updated Feb. 10, 2016, 12:22 a.m.)


Review request for sentry and Gregory Chanan.


Changes
-------

Updated based on comments


Repository: sentry


Description
-------

Group names are case sensitive, handle them as such Generic model


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java
 e1c15fa5304b553f69ef4d7e5053d587efb92ae5 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericPolicyProcessor.java
 78d38473f938b1aa2b357dd858b00f833147ee5d 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryShellSolr.java
 f1a87a83a97f3e2d7d06462abaaf4b983ab17677 

Diff: https://reviews.apache.org/r/43242/diff/


Testing
-------

Existing tests pass, and also added a new test.


Thanks,

Sravya Tirukkovalur

Reply via email to