You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Brock Noland <br...@cloudera.com> on 2013/10/10 00:11:02 UTC
Re: Review Request 14563: SENTRY-39 Rename class in "search" module to Search
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14563/#review26844
-----------------------------------------------------------
Ship it!
Ship It!
- Brock Noland
On Oct. 9, 2013, 10:09 p.m., Gregory Chanan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14563/
> -----------------------------------------------------------
>
> (Updated Oct. 9, 2013, 10:09 p.m.)
>
>
> Review request for sentry and Brock Noland.
>
>
> Bugs: SENTRY-39
> https://issues.apache.org/jira/browse/SENTRY-39
>
>
> Repository: sentry
>
>
> Description
> -------
>
> Straightforward renames, requires SENTRY-34.
>
>
> Diffs
> -----
>
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/Collection.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SearchConstants.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SearchModelAction.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SearchModelAuthorizable.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SolrConstants.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SolrModelAction.java PRE-CREATION
> sentry-core/sentry-core-model-search/src/main/java/org/apache/sentry/core/model/search/SolrModelAuthorizable.java PRE-CREATION
>
> Diff: https://reviews.apache.org/r/14563/diff/
>
>
> Testing
> -------
>
> Ran unit tests (not end to end tests)
>
>
> Thanks,
>
> Gregory Chanan
>
>