----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43242/#review118495 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryShellSolr.java (line 131) <https://reviews.apache.org/r/43242/#comment179748> Can we just remove this line, since the flag is no longer needed. - Hao Hao 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 > >
