----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63878/#review191657 -----------------------------------------------------------
Ship it! Ship It! - kalyan kumar kalvagadda On Nov. 21, 2017, 7:08 p.m., Hrishikesh Gadre wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63878/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2017, 7:08 p.m.) > > > Review request for sentry, kalyan kumar kalvagadda and Sergio Pena. > > > Bugs: SENTRY-1480 > https://issues.apache.org/jira/browse/SENTRY-1480 > > > Repository: sentry > > > Description > ------- > > SENTRY-1475 upgraded the Sentry/SOLR plugin to use the latest version of SOLR > (7.1.0) which provides pluggable authorization framework. With this, the way > admin privileges are defined is changed. This patch provides a command-line > tool to migrate existing privileges to this new model. It supports both (a) > file based Sentry and (b) Sentry service configuration. > > > Diffs > ----- > > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/PolicyFileConstants.java > 6b625ffb61159fa50a7d556762fd08f33e873c40 > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/PolicyFiles.java > ca2de7a165f0c8c37efe2b0c1ab3858addc15acf > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/StrictStringTokenizer.java > PRE-CREATION > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/Version.java > PRE-CREATION > > sentry-policy/sentry-policy-common/src/main/java/org/apache/sentry/policy/common/PrivilegeUtils.java > 6628a2f3cbb338542a6f02a0387b6fd7a0e092dc > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/PermissionsMigrationToolCommon.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/PermissionsMigrationToolSolr.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestPermissionsMigrationToolSolr.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/63878/diff/2/ > > > Testing > ------- > > All unit tests are passing. > Added a unit test to verify this migration tool. > > > Thanks, > > Hrishikesh Gadre > >
