----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34756/#review102125 -----------------------------------------------------------
Also, some of the items here are fixed with HIVE-10875, be sure to comment that we might have to unignore the tests when we upgrade the hive dependency. - Sravya Tirukkovalur On Oct. 9, 2015, 4:06 a.m., Anne Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34756/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2015, 4:06 a.m.) > > > Review request for sentry, Li Li, Lenni Kuff, and Sravya Tirukkovalur. > > > Bugs: SENTRY-748 > https://issues.apache.org/jira/browse/SENTRY-748 > > > Repository: sentry > > > Description > ------- > > Add new test case to validate complex view privilege: view by union tables, > view from another view, view with subquery and view by joining tables; > > Resent the cr. > > > Diffs > ----- > > sentry-tests/sentry-tests-hive/pom.xml > 7ee5378fdd2af383c4dc9b7f692d60b9c1173038 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/34756/diff/ > > > Testing > ------- > > https://builds.apache.org/job/PreCommit-SENTRY-Build/608/, seems like tests > all passed now. > > > Thanks, > > Anne Yu > >
