[jira] [Comment Edited] (IGNITE-6053) IgniteCache.clear clears local caches with same names on all server nodes

2017-09-01 Thread Roman Shtykh (JIRA)

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

Roman Shtykh edited comment on IGNITE-6053 at 9/2/17 3:14 AM:
--

[~ntikhonov] Thanks for checking it again! Btw, why would it be better to use 
the public pool for this method? I am just curious -- other methods in 
`GridCacheAdapter` use the system pool.
Anyway, I changed it to the public pool for tests.

I ran TC tests now. Looks good to me.
If you are ok with it, I will merge.


was (Author: roman_s):
[~ntikhonov] Thanks for checking it again! Btw, why would it be better to use 
the public pool for this method? I am just curious -- other methods in 
`GridCacheAdapter` use the system pool.

I ran TC tests now. Looks good to me.

> IgniteCache.clear clears local caches with same names on all server nodes
> -
>
> Key: IGNITE-6053
> URL: https://issues.apache.org/jira/browse/IGNITE-6053
> Project: Ignite
>  Issue Type: Bug
>Affects Versions: 1.7
>Reporter: Evgenii Zhuravlev
>Assignee: Roman Shtykh
> Fix For: 2.3
>
>
> Clear on LOCAL cache should clear cache on the current node only, not on all 
> nodes, that have local caches with same names.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Comment Edited] (IGNITE-6053) IgniteCache.clear clears local caches with same names on all server nodes

2017-09-01 Thread Nikolay Tikhonov (JIRA)

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

Nikolay Tikhonov edited comment on IGNITE-6053 at 9/1/17 11:43 AM:
---

[~roman_s],
I've looked at code and have only one comment, let's will execute clear closure 
in public pool instead of system pool. For it need to pass false to localSafe 
method. {{ctx.closures().callLocalSafe(..., false)}}
Also I don't see a runs on TC. Are you sure that you triggered this suites 
[link 
TC|https://ci.ignite.apache.org/project.html?projectId=Ignite20Tests&branch_Ignite20Tests=pull%2F2443%2Fhead]?
BTW Thank you for your contribution!


was (Author: ntikhonov):
[~roman_s],
I've looked at code and have only one comment, let's will execute clearTask in 
public pool instead of system pool. For it need to pass false to localSafe 
method. {{ctx.closures().callLocalSafe(..., false)}}
Also I don't see a runs on TC. Are you sure that you triggered this suites 
[link 
TC|https://ci.ignite.apache.org/project.html?projectId=Ignite20Tests&branch_Ignite20Tests=pull%2F2443%2Fhead]?
BTW Thank you for your contribution!

> IgniteCache.clear clears local caches with same names on all server nodes
> -
>
> Key: IGNITE-6053
> URL: https://issues.apache.org/jira/browse/IGNITE-6053
> Project: Ignite
>  Issue Type: Bug
>Affects Versions: 1.7
>Reporter: Evgenii Zhuravlev
>Assignee: Roman Shtykh
> Fix For: 2.3
>
>
> Clear on LOCAL cache should clear cache on the current node only, not on all 
> nodes, that have local caches with same names.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)