> On June 3, 2016, 11:14 p.m., Sravya Tirukkovalur wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java, > > line 2183 > > <https://reviews.apache.org/r/48195/diff/1/?file=1405582#file1405582line2183> > > > > As the problem is with grant statement not treating the prefix > > correctly, can we rename the test to testGrantPrivilegeWithDbPrefix instead?
I felt that the grant actually works, as the hdfs ACL's are proper after grant. Its only when you do a show you release this is a problem. - Rahul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48195/#review136130 ----------------------------------------------------------- On June 3, 2016, 1:25 a.m., Rahul Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48195/ > ----------------------------------------------------------- > > (Updated June 3, 2016, 1:25 a.m.) > > > Review request for sentry and Sravya Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > SENTRY-858: Database prefix is not honoured when executing grant statement > > > Diffs > ----- > > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java > a7c0a0c4bac26e9f6c4dcb1e2dc61debd6007b7c > > Diff: https://reviews.apache.org/r/48195/diff/ > > > Testing > ------- > > > Thanks, > > Rahul Sharma > >
