----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63687/#review190627 -----------------------------------------------------------
sentry-core/sentry-core-common/src/test/java/org/apache/sentry/core/common/utils/TestKeyValue.java Lines 58 (patched) <https://reviews.apache.org/r/63687/#comment268129> Can you add test case to check value is also case insensitive? - Na Li On Nov. 9, 2017, 4:34 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63687/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2017, 4:34 p.m.) > > > Review request for sentry. > > > Bugs: sentry-2039 > https://issues.apache.org/jira/browse/sentry-2039 > > > Repository: sentry > > > Description > ------- > > Return the equalsIgnoreCase() to the KeyValue.equals() method (removed by > SENTRY-999) > > > Diffs > ----- > > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/KeyValue.java > 4e944e5f2605459ac4166a784bb1d50362509221 > > sentry-core/sentry-core-common/src/test/java/org/apache/sentry/core/common/utils/TestKeyValue.java > ca44a245dc9abf003eaae13d0ebbfb1ef7eff29a > > sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/TestCommonPrivilege.java > 3f60b1901976d2236db0814fd63114c9598e8215 > > > Diff: https://reviews.apache.org/r/63687/diff/2/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >