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

Josh Wills commented on CRUNCH-122:
-----------------------------------

+1, please commit.
                
> Reduce visibility of implementation classes
> -------------------------------------------
>
>                 Key: CRUNCH-122
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-122
>             Project: Crunch
>          Issue Type: Improvement
>    Affects Versions: 0.4.0
>            Reporter: Matthias Friedrich
>            Assignee: Matthias Friedrich
>            Priority: Trivial
>         Attachments: CRUNCH-122-Make-impl-classes-package-private.patch, 
> CRUNCH-122-v2.patch, CRUNCH-122-v3.patch
>
>
> There are a few implementation classes that are used only from within their 
> home package. Thus, they should be package-private instead of public, which 
> also hides them from javadoc. Most of these classes aren't from our set of 
> "published" packages anyway but it's good style to hide them if we can. We 
> can also make inner DoFns private if there is a static factory method for 
> them.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to