sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sravya Tirukkovalur (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SENTRY-200) Remove sentry-provider dependencies on hive
Date Fri, 16 May 2014 10:20:06 GMT

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

Sravya Tirukkovalur commented on SENTRY-200:
--------------------------------------------

+1, LGTM. Will commit if the tests pass.

> Remove sentry-provider dependencies on hive 
> --------------------------------------------
>
>                 Key: SENTRY-200
>                 URL: https://issues.apache.org/jira/browse/SENTRY-200
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.4.0
>            Reporter: Lenni Kuff
>            Assignee: Prasad Mujumdar
>            Priority: Blocker
>             Fix For: 1.4.0
>
>         Attachments: SENTRY-200.2.patch
>
>
> sentry-provider now seems to have some dependencies on Hive. Only the binding layer should
have these dependencies. It seems some of this may have crept in as part of the SentrySchemaTool
work:
> For example:
> {code}
> sentry-provider/sentry-provider-db/pom.xml:      <artifactId>hive-metastore</artifactId>
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreSchemaInfo.java:import
org.apache.hadoop.hive.metastore.HiveMetaException;
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreSchemaInfo.java:
  * Find the directory of metastore scripts
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java:import
org.apache.hadoop.hive.metastore.api.MetaException;
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java:
   Connection metastoreConn = getConnectionToMetastore(true);
> {code}



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

Mime
View raw message