[SENTRY-1047] Use existing validators in SentryShellSolr

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.7.0
• Fix Version/s: 1.7.0
• Component/s: Service, Solr Plugin
• Assignee: Gregory Chanan
• Reporter: Gregory Chanan
• Created: 03/Feb/16 01:38
• Updated: 03/Feb/16 20:45
• Resolved: 03/Feb/16 20:45

Description

Right now, following the pattern of the Hive shell, the Solr shell manually implements validators, i.e.:
https://github.com/apache/incubator-sentry/blob/597a3cdd319be84f2417c96d24db01553f264551/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SolrTSentryPrivilegeConvertor.java#L115-L130

It would be better if we automatically used the existing validators, so they don't need to be implemented in multiple places.

Issue Links

• is related to: SENTRY-1049 Use existing validators in hive shell. Automatically for external generic services?
• links to: Review Request