> On Jan. 7, 2016, 8:29 a.m., Lenni Kuff wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java, > > line 520 > > <https://reviews.apache.org/r/41662/diff/2/?file=1181542#file1181542line520> > > > > Add a test case for truncating an unpartitioned table if one does not > > already exist.
the test case for truncating an unpartitioned table already exists: testTruncateTable > On Jan. 7, 2016, 8:29 a.m., Lenni Kuff wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java, > > line 276 > > <https://reviews.apache.org/r/41662/diff/2/?file=1181541#file1181541line276> > > > > Can you add a precondition check to ensure we have the expected number > > of children? Will we have this number of children for unpartitioned tables > > as well? Sure. Yes. - Li ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41662/#review113219 ----------------------------------------------------------- On Jan. 5, 2016, 4:52 a.m., Li Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41662/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 4:52 a.m.) > > > Review request for sentry, Anne Yu, Hao Hao, Lenni Kuff, and Sravya > Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > The SemanticException happened in HiveAuthzBinding.authorize method, which is > caused by the empty outputHierarchy list. As there is no partition in the > test tbl, the writeEntity set returned from Hive is empty. That's why the > outputHierarchy list is also empty. To resolve it, we should manually add > db.tbl in the outputHierarchy which will be validated in the > hiveAuthzBinding.authorize method. > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java > 994af8a04414f5093a0d5546744d6da727b69db4 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java > 56776db7c463e1f27cd39baa9e4c9d9709c53925 > > Diff: https://reviews.apache.org/r/41662/diff/ > > > Testing > ------- > > add test case for: > empty partitioned table and NOT empty partitioned table > user with permission and user WITHOUT permission > > > Thanks, > > Li Li > >
