> On Feb. 5, 2016, 8:36 a.m., Lenni Kuff wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericPolicyProcessor.java, > > line 481 > > <https://reviews.apache.org/r/43242/diff/1/?file=1235971#file1235971line481> > > > > Are group names really case-insensitive? Linux groups are usually > > case-insensitive, AD groups are as well. > > > > While you are here can you fix toTrimedLower to spell "trimmed" > > properly?
I did some reading about it, and it looks like AD and OpenLap are by default case insensitive, but can be configured otherwise. But looks like Hadoop is case sensitive :-). So not sure which standard we should follow. Thoughts? - Sravya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43242/#review117990 ----------------------------------------------------------- On Feb. 5, 2016, 6:21 p.m., Sravya Tirukkovalur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43242/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2016, 6:21 p.m.) > > > Review request for sentry and Gregory Chanan. > > > Repository: sentry > > > Description > ------- > > Group names are case insensitive, handle them as such in list apis. > > > Diffs > ----- > > > 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 > ------- > > Test already present, tweaked it to use mixed case. > > > Thanks, > > Sravya Tirukkovalur > >
