> 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? > > Rahul Sharma wrote: > 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.
Ah, interesting. I checked and hive permissions only work on default after this and not the original db "db1". I would guess HDFS works as the delta we are propagating is correct but by the time it is persisted it is using the "default" db. In that case, I am ok with this naming. - Sravya ----------------------------------------------------------- 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 > >
