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

ASF GitHub Bot commented on IGNITE-3585:
----------------------------------------

GitHub user isapego opened a pull request:

    https://github.com/apache/ignite/pull/1713

    IGNITE-3585 CPP: Container-based methods now accept iterators.

    

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

    $ git pull https://github.com/gridgain/apache-ignite ignite-3585

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

    https://github.com/apache/ignite/pull/1713.patch

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

    This closes #1713
    
----
commit de19ce621ba6d82c759a1e96d0b8b610bab9d4fb
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-24T15:48:38Z

    IGNITE-3585: Implemented for Cursor::GetAll

commit 7f8e735a8a52dc96c89bedd8be00fab2bafcb4b4
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-27T11:45:16Z

    IGNITE-3585: Fixed tests

commit 5b2c2b6218213e8e2a5149c42052095eacc82107
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-29T17:58:15Z

    IGNITE-3585: Implemented for ContainsKeys

commit c0dba0e0109cc73cf3c9757c155d46bacf3a9c72
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-30T17:20:44Z

    IGNITE-3585: Implemented for PutAll

commit be8af85892b17168364e3671c310a20f7d49afa2
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-31T11:56:24Z

    IGNITE-3585: Implemented GetAll

commit 3347971262d924b07a298eab020a1cc8cb89006a
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-31T12:10:58Z

    IGNITE-3585: Implemented for LocalEvict

commit 046d848aad1aa573a66468fb13efd279adbadb88
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-31T12:25:25Z

    IGNITE-3585: Implemented for ClearAll and LocalClearAll

commit 1e11a6209aa9231e78c318afc70b429c85f9a927
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-31T12:33:52Z

    IGNITE-3585: Implemented for RemoveAll

commit 28dc1b966381480e63aa8cef27a5e3fafdbda46d
Author: Igor Sapego <isap...@gridgain.com>
Date:   2017-03-31T12:54:40Z

    Merge remote-tracking branch 'upstream/master' into ignite-3585

----


> CPP: Rework methods that accept containers so any container could be used 
> with them.
> ------------------------------------------------------------------------------------
>
>                 Key: IGNITE-3585
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3585
>             Project: Ignite
>          Issue Type: Task
>          Components: platforms
>    Affects Versions: 1.6
>            Reporter: Igor Sapego
>            Assignee: Igor Sapego
>              Labels: cpp
>             Fix For: 2.0
>
>
> There are methods that could possibly be used with any container but by some 
> reason they only accept arguments of a single specific container type. It can 
> be non convenient for user and bad from the performance point of view if user 
> uses other container type to store values. Consider using iterators 
> (preferably) or template type for the container.
> Methods of interest:
> - {{QueryCursor::GetAll}}
> - {{Cache::ContainsKeys}}
> - {{Cache::GetAll}}
> - {{Cache::PutAll}}
> - {{Cache::LocalEvict}}
> - {{Cache::ClearAll}}
> - {{Cache::LocalClearAll}}
> - {{Cache::RemoveAll}}.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to