sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brock Noland (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (SENTRY-105) Remove db specific name from sentry service package
Date Thu, 06 Feb 2014 23:13:23 GMT

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

Brock Noland edited comment on SENTRY-105 at 2/6/14 11:13 PM:
--------------------------------------------------------------

The attached patch does the following:

* Breaks up the policy specific parts of Sentry service and the Policy Service. The expectation
is that in the future Sentry Service might do more than just policy stuff. While it doesn't
move the classes out of the policy-db module, it does break them apart from a package perspective.
* Changes TStatus to an int as opposed to enum since enums are not backwards compatible.
* Allows SentryService to add a configurable set of "processors" one of them being SentryPolicyStoreProcessor
* Breaks up thrift defs and moves them to a standard location src/main/resources


was (Author: brocknoland):
The attached patch does the following:

* Breaks up the policy specific parts of Sentry service and the Policy Service. The expectation
is that in the future Sentry Service might do more than just policy stuff. While it doesn't
move the classes out of the policy-db module, it does break them apart from a package perspective.
* Changes TStatus to an int as opposed to enum since enums are not backwards compatible.
* Allows SentryService to add a configurable set of "processors" one of them being SentryPolicyStoreProcessor

> Remove db specific name from sentry service package
> ---------------------------------------------------
>
>                 Key: SENTRY-105
>                 URL: https://issues.apache.org/jira/browse/SENTRY-105
>             Project: Sentry
>          Issue Type: Sub-task
>    Affects Versions: db_policy_store
>            Reporter: Brock Noland
>            Assignee: Brock Noland
>         Attachments: SENTRY-105.patch
>
>
> Right now "SentryService" lives in sentry-provider-db. Ultimately we will probably want
it to have a separate module separate from the DB stuff. However, for now let's just change
the package name.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message