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

Denis Magda commented on IGNITE-1481:
-------------------------------------

Hi, [~kcheng.mvp]

I've reviewed you changes, thanks for the contribution!

1) Please address my comments left directly in PR
https://github.com/apache/ignite/pull/449/files

2) After all my comments are applied please check all test suites on TC. Use 
'Run All' button for that. The reason is that you made changes in one of the 
most widely used classes and this can affect other functionality

> It is possible to configure local cache with affinity function
> --------------------------------------------------------------
>
>                 Key: IGNITE-1481
>                 URL: https://issues.apache.org/jira/browse/IGNITE-1481
>             Project: Ignite
>          Issue Type: Bug
>          Components: newbie
>            Reporter: Andrey Gura
>            Assignee: kcheng.mvp
>
> {{LOCAL}} cache always uses {{GridCacheProcessor.LocalAffinityFunction}}.
> Durign configuration of local cache it is possible to pass other affinity 
> function to cache configuration. All cache operations will work correctly, 
> but user can obtain affinity manager that internally uses incorrect affinity 
> function ({{GridCacheContext.cctx.affinity().aff.aff}}).
> Expected behaviour: {{GridCacheContext.cctx.affinity().aff.aff}} always 
> refers to {{GridCacheProcessor.LocalAffinityFunction}} for {{LOCAL}} caches.



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

Reply via email to