You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Christopher Tubbs (JIRA)" <ji...@apache.org> on 2018/03/19 20:25:00 UTC

[jira] [Resolved] (ACCUMULO-4820) Cleanup code for 2.0

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

Christopher Tubbs resolved ACCUMULO-4820.
-----------------------------------------
    Resolution: Fixed

> Cleanup code for 2.0
> --------------------
>
>                 Key: ACCUMULO-4820
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4820
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 2.0.0
>            Reporter: Michael Miller
>            Assignee: Michael Miller
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 2.0.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Running IntelliJ code inspect picks up a lot of minor code clean up fixes that we would be nice to fix sooner rather than later.
> Java 5 Updates 
> - unnecessary boxing
> - use foreach where possible
> Java 7 Updates (these should definitely be fixed)
> - Explicit type can be replaced with <> (aka diamond operator)
> - Identical catch branches in try 
> - try finally replaceable with try with resources
> Java 8 Updates (these would be nice, but maybe some prefer older way?)
> - Replace anonymous types with lambda
> - Replace code with new single Map method call
> - Replace Collections.sort () with List.sort()
> Other Misc performance Issues picked up by the inspector.  I think these should definitely be fixed but perhaps a sub ticket



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)