[jira] [Commented] (IGNITE-2394) Cache loading from storage is called on client nodes

["Alpet Tekinalp (JIRA)" <ji...@apache.org>]

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

Alpet Tekinalp commented on IGNITE-2394:
----------------------------------------

Hi.

I want to work on that bug. Based on [~vkulichenko]'s comment I IgniteCacheProxy.loadCache that it checks cache type and if cache is local it calls cache().localLoadCache. 

So when you call either client or server cache is only loaded locally. Is that the correct way to imlement?

I appreciate your comments.

> Cache loading from storage is called on client nodes
> ----------------------------------------------------
>
>                 Key: IGNITE-2394
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2394
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 1.5.0.final
>            Reporter: Denis Magda
>            Priority: Critical
>              Labels: newbie
>             Fix For: 1.6
>
>         Attachments: LocalLoadTest.java, master_baa1312_IGNITE-2394.patch
>
>
> If to call cache.loadCache(...) then the loading from a storage will happen on all the nodes including client nodes.
> However, client nodes must be filtered out.
> If to be more precise at least this place of the code has to be modified
> {noformat}
>     IgniteInternalFuture<?> globalLoadCacheAsync(@Nullable IgniteBiPredicate<K, V> p, @Nullable Object... args)
>         throws IgniteCheckedException {
>         ClusterGroup nodes = ctx.kernalContext().grid().cluster().forCacheNodes(ctx.name());
> {noformat}
> where forDataNodes() has to be used instead of forCacheNodes().
> Also additional tests have to be added.
> Discussion on the user list:
> http://apache-ignite-users.70518.x6.nabble.com/Loadcache-behavior-tp2571.html



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