> On Aug. 8, 2018, 8:07 p.m., Na Li wrote: > > can you add test case for this change?
The new patch has a new test case for this. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67888/#review206994 ----------------------------------------------------------- On Aug. 16, 2018, 4:18 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67888/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2018, 4:18 p.m.) > > > Review request for sentry and Arjun Mishra. > > > Bugs: sentry-2308 > https://issues.apache.org/jira/browse/sentry-2308 > > > Repository: sentry > > > Description > ------- > > Do not allow CREATE privileges on a table. There is no use case for it. > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java > 2c662e55382adc36abd18b6b6b82aa7a0b2f210b > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java > beca2f8f3fd03953d5652337dee7e69313f098a3 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestAllowedGrantPrivileges.java > PRE-CREATION > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestDescribeMetadataPrivileges.java > 54f4f2fdba964ebd1e610a87dfaa370721741af2 > > > Diff: https://reviews.apache.org/r/67888/diff/3/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >
