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

Shreepadma Venugopalan commented on SENTRY-143:
-----------------------------------------------

I looked over the patch, seems alright. The patch is too large for a through 
review, but I guess that's not the goal of this review.

> Merge db_policy_store branch into master
> ----------------------------------------
>
>                 Key: SENTRY-143
>                 URL: https://issues.apache.org/jira/browse/SENTRY-143
>             Project: Sentry
>          Issue Type: Sub-task
>            Reporter: Shreepadma Venugopalan
>            Assignee: Brock Noland
>         Attachments: SENTRY-143-no-thrift.patch, SENTRY-143.patch
>
>
> We'll do this as a patch to ensure we don't get a merge commit on master. 
> It's important to note that since db_policy_store uses java 7 and we haven't 
> got precommit tests using java 7 yet, this will disable precommit tests, 
> which I think is fine, for now.



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

Reply via email to