----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22780/#review46212 -----------------------------------------------------------
Ship it! Looks good Joey.. Thanks !! - Arun Suresh On June 19, 2014, 6:34 p.m., Joey Echeverria wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22780/ > ----------------------------------------------------------- > > (Updated June 19, 2014, 6:34 p.m.) > > > Review request for sentry. > > > Bugs: SENTRY-199 > https://issues.apache.org/jira/browse/SENTRY-199 > > > Repository: sentry > > > Description > ------- > > Latest patch that adds a --import option to config-tool. > > > Diffs > ----- > > sentry-binding/sentry-binding-hive/pom.xml 4974108 > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java > 370c00d > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/SentryMain.java > 58233e0 > > sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/SimpleFileProviderBackend.java > 61eb694 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImport.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/22780/diff/ > > > Testing > ------- > > Added automated test case as part of the Hive end-to-end tests. > > Manually tested on a cluster running Sentry trunk. > > > Thanks, > > Joey Echeverria > >
