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

Sergey Evdokimov commented on IGNITE-59:
----------------------------------------

Text from mail conversation:

Sergey E: Will we support lockAsync() on IgniteCache? Should I remove all tests 
that check lockAsync() methods?
Dmitriy: We should support it through our standard async support. I will take a 
closer look at the design and will provide more detail.


Sergey E: Will be exists GridCacheEntry.isLocked() , 
GridCacheEntry.isLockedByThread() ? If true we need the method to get entry by 
key on IgniteCache (analog of GridCacheProjection.entry(K key)). 
Dmitriy: These methods will not exist on GridCacheEntry.


Sergey E: In current IgniteCache interface lock() returns 
java.util.concurrent.locks.Lock . May be 'lock()' should return something like 
CacheLock, that extends java.util.concurrent.locks.Lock and contains additional 
methods like "isLocked()", "isLockedByThread()". Thoughts?   Now IgniteCache 
contains "isLocked(key)", "isLockedByThread(keys)" methods.


Sergey E: Why lockAll(Set) take a Set, may be it should take Collection ?

> Support lock, lockAll
> ---------------------
>
>                 Key: IGNITE-59
>                 URL: https://issues.apache.org/jira/browse/IGNITE-59
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: cache
>            Reporter: Semen Boikov
>            Assignee: Sergey Evdokimov
>             Fix For: sprint-1
>
>
> Implement methods on IgniteCache: 'Lock lock(K key)', 'Lock lockAll(Set<? 
> extends K> keys)', note: in new design method return Lock instance.



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

Reply via email to