incubator-crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gabriel Reid (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (CRUNCH-149) Remove unnecessary "throws IOException" on CrunchTool's constructor
Date Thu, 24 Jan 2013 20:59:13 GMT

     [ https://issues.apache.org/jira/browse/CRUNCH-149?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Gabriel Reid resolved CRUNCH-149.
---------------------------------

       Resolution: Fixed
    Fix Version/s: 0.5.0
         Assignee: Gabriel Reid  (was: Josh Wills)

Pushed to master.

Thanks for this Dave! I thought we'd gotten rid of all of those unnecessary throws clauses,
but apparently there are (or at least were) still some lurking around. 
                
> Remove unnecessary "throws IOException" on CrunchTool's constructor
> -------------------------------------------------------------------
>
>                 Key: CRUNCH-149
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-149
>             Project: Crunch
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Dave Beech
>            Assignee: Gabriel Reid
>            Priority: Trivial
>             Fix For: 0.5.0
>
>         Attachments: CRUNCH-149.patch
>
>
> CrunchTool's no-arg constructor throws IOException but doesn't seem to need to. Can we
remove this? At the moment it forces subclasses to have an explicit constructor even if you
just want to use the implicit one. 

--
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

Mime
View raw message