You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Pavel Tupitsyn (JIRA)" <ji...@apache.org> on 2017/04/05 12:00:45 UTC
[jira] [Comment Edited] (IGNITE-4899) .NET: Review Dictionary usage
in APIs
[ https://issues.apache.org/jira/browse/IGNITE-4899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15956725#comment-15956725 ]
Pavel Tupitsyn edited comment on IGNITE-4899 at 4/5/17 11:59 AM:
-----------------------------------------------------------------
{{GetAll}} reads data from pooled memory using pooled stream. Returning lazy {{IEnumerable}} will capture these pooled things and may lead to undefined behavior. We should not do this.
However, dictionary is still inefficient when it is used as a collection.
was (Author: ptupitsyn):
{{GetAll}} reads data from pooled memory using pooled stream. Returning lazy {{IEnumerable}} will capture these pooled things and may lead to undefined behavior. We should not do this.
> .NET: Review Dictionary usage in APIs
> -------------------------------------
>
> Key: IGNITE-4899
> URL: https://issues.apache.org/jira/browse/IGNITE-4899
> Project: Ignite
> Issue Type: Improvement
> Affects Versions: 2.0
> Reporter: Pavel Tupitsyn
> Assignee: Pavel Tupitsyn
> Labels: .NET, breaking-api
> Fix For: 2.0
>
>
> We have replaced {{IDictionary}} with {{IEnumerable<KeyValuePair<TK, TV>>}} in {{ICacheStore}}, let's do the same for other APIs like {{ICache.GetAll}}.
> Reading GetAll results into a dictionary is inefficient in case when user only needs to iterate over results (unneeded allocation and hashing).
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)