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)