accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Miller (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (ACCUMULO-4820) Cleanup code for 2.0
Date Fri, 16 Feb 2018 19:46:00 GMT

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

Michael Miller updated ACCUMULO-4820:
-------------------------------------
    Description: 
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

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


> 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
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> 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)

Mime
View raw message