----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14165/#review26307 -----------------------------------------------------------
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java <https://reviews.apache.org/r/14165/#comment51394> Shouldn't deprecatedConfigs be a map of the deprecated config's name to the one that' currently in use? sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestHiveAuthzBindings.java <https://reviews.apache.org/r/14165/#comment51395> I don't see sentry-deprecated-site.xml in this patch. Does this patch assume SENTRY-16 is available? - Shreepadma Venugopalan On Sept. 17, 2013, 6:27 p.m., Gregory Chanan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14165/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2013, 6:27 p.m.) > > > Review request for sentry and Shreepadma Venugopalan. > > > Bugs: SENTRY-4 > https://issues.apache.org/jira/browse/SENTRY-4 > > > Repository: sentry > > > Description > ------- > > SENTRY-4: Rename Configuration properties that mention hive but are sentry > related > > As discussed in the JIRA, does the following: > 0) add new sentry.* properties, as discussed in SENTRY-14 > 1) remove deprecation code around hive.access.* properties > 2) add deprecation code around hive.sentry.* properties > 3) warn about deprecated configs only if they actually end up being used > > I'm a bit unsure of spacing (how many spaces is a tab? I used two in most > places -- perhaps something to add to the how to contribute page?) > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBinding.java > 5190ba6 > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java > bfd58fa > > sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestHiveAuthzBindings.java > 20d4e8f > > sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestHiveAuthzConf.java > de59546 > sentry-binding/sentry-binding-hive/src/test/resources/access-site.xml > 1936c21 > sentry-binding/sentry-binding-hive/src/test/resources/sentry-site.xml > beef40d > sentry-tests/src/test/resources/access-site.xml e4de05a > sentry-tests/src/test/resources/sentry-site.xml de0c9cf > > Diff: https://reviews.apache.org/r/14165/diff/ > > > Testing > ------- > > Ran the unit tests and they were successful: > mvn test -Pdownload-hadoop > > > Thanks, > > Gregory Chanan > >
