----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46058/ -----------------------------------------------------------
(Updated April 14, 2016, 2:51 a.m.) Review request for sentry, shen guoquan, Sravya Tirukkovalur, and Vamsee Yarlagadda. Changes ------- Moved ini test files to src/test/resources. Repository: sentry Description ------- Updates guoquan's original patch to the latest Sentry API's and layout. In particular the differences from the original patch are: 1) Convert to the latest Generic APIs 2) Is a tool like the SolrSentryShell supporting similar flags 3) No longer builds a (SolrAuthz)Binding, only uses the file provider backend and the validators. Thus, we don't require the binding which is tricky to get right anyway (the binding really runs in the server...are we convinced we have all the right system properties, etc to get it to work)? 4) because of 3), doesn't pass the policy file via the configuration file, takes it on the command line. We need the configuration file to specify the service info; having both the policy file and the service info in there means users would have to create a configuration file just for use of the tool. 5) Adds some additional compatibility checking around the fact that role names are lower cased in the service but at least the solr backend supported cased role names. Like the SolrSentryShell, this should probably live in sole solr-specific client module, but since that doesn't exist yet, I put it in the sentry-provider-db, which is the same place the SolrSentryShell exists. Diffs (updated) ----- sentry-provider/sentry-provider-db/pom.xml 8d9fa55723541f697171d8958e250f17f8159ddf sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryConfigToolCommon.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryConfigToolSolr.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SolrTSentryPrivilegeConvertor.java e2dfdf13d0c5f665080fb1c35da8b1cb648019df sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryConfigToolSolr.java PRE-CREATION sentry-provider/sentry-provider-db/src/test/resources/solr_case.ini PRE-CREATION sentry-provider/sentry-provider-db/src/test/resources/solr_config_import_tool.ini PRE-CREATION sentry-provider/sentry-provider-db/src/test/resources/solr_invalid.ini PRE-CREATION Diff: https://reviews.apache.org/r/46058/diff/ Testing ------- Ran the unit tests successfully. Thanks, Gregory Chanan
