ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-2273) Arrays.asList() should not be used for 1- and 2- elements.
Date Wed, 27 Jan 2016 14:20:40 GMT

    [ https://issues.apache.org/jira/browse/IGNITE-2273?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15119285#comment-15119285

ASF GitHub Bot commented on IGNITE-2273:

GitHub user ilantukh opened a pull request:




You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ilantukh/ignite ignite-2273

Alternatively you can review and apply these changes as the patch at:


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #429
commit 0e6e55bce1567430caf47bfaf7a498f9172031ff
Author: Ilya Lantukh <ilantukh@gridgain.com>
Date:   2016-01-27T14:15:41Z

    ignite-2273 : Alternative to Arrays.asList() optimized for 1 and 2 element arrays.

commit 4ec95281af610334296d258d6b8b8d1f533c11e0
Author: Ilya Lantukh <ilantukh@gridgain.com>
Date:   2016-01-27T14:19:47Z

    ignite-2273 : reverted Arrays.asList() replacement in examples.


> Arrays.asList() should not be used for 1- and 2- elements.
> ----------------------------------------------------------
>                 Key: IGNITE-2273
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2273
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: general
>    Affects Versions: ignite-1.4
>            Reporter: Vladimir Ozerov
>            Assignee: Ilya Lantukh
>             Fix For: 1.6
> There are several places in code where Arrays.asList is used as follows:
> 1) For a single element. These places should be replaced with Collections.singleton()
to avoid unnecessary array allocations.
> 2) For two elements. Here we should create our own "doubled-singleton" implementation.
> Some of these places are located on hot code paths (e.g. cache PUT).

This message was sent by Atlassian JIRA

View raw message