You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by "SentryQA (JIRA)" <ji...@apache.org> on 2014/07/24 10:39:38 UTC

[jira] [Commented] (SENTRY-362) When sentry integrate into solr, the create instance of backend needs configure parameters from solrAuthzConf not hadoopConf

    [ https://issues.apache.org/jira/browse/SENTRY-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14072993#comment-14072993 ] 

SentryQA commented on SENTRY-362:
---------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12657565/SENTRY-362.patch against master.

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} failed to build with patch (exit code 137)

Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-SENTRY-Build/231/console

This message is automatically generated.

> When sentry integrate into solr, the create instance of backend needs configure parameters from solrAuthzConf not hadoopConf
> ----------------------------------------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-362
>                 URL: https://issues.apache.org/jira/browse/SENTRY-362
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.3.0
>            Reporter: guoquanshen
>            Assignee: guoquanshen
>         Attachments: SENTRY-362.patch
>
>
> whatever Sentry Solr plugin integration with db store or file store, the create instance of DBProviderBackend of FIleProviderBackend should get the configuration from sentry-site.xml not hadoop configuration. The following code isn't correct:
> {code:title=SolrAuthzBinding.java}
>     Configuration conf = getConf();
>     providerBackend =(ProviderBackend) providerBackendConstructor.newInstance(new Object[] {conf, resourceName});
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)