[jira] [Commented] (IGNITE-2263) Get rid of wrapping views where possible.

[Sergi Vladykin (JIRA)]

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

Sergi Vladykin commented on IGNITE-2263:
----------------------------------------

I think we should start with F.view(). Replacement should look IMO as simple as:
{code}
    public static <T> Collection<T> filter(Collection<? extends T> src, 
Collection<T> dst, IgnitePredicate<? super T> p) {
        if (!isEmpty(src) && !isAlwaysFalse(p)) {
            for (T e : src)
                dst.add(e);
        }
        return dst;
    }
{code}


> Get rid of wrapping views where possible.
> -----------------------------------------
>
>                 Key: IGNITE-2263
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2263
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: general
>    Affects Versions: ignite-1.4
>            Reporter: Vladimir Ozerov
>            Assignee: Vladimir Ozerov
>            Priority: Critical
>             Fix For: 1.6
>
>
> We have about ~50-100 usages of things like F.view or F.viewReadOnly. In lots 
> cases it is not necessary, adds garbage, but doesn't add any value. 
> Need to revisit these places.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)