[
https://issues.apache.org/jira/browse/IGNITE-6043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16182215#comment-16182215
]
Andrey Kuznetsov commented on IGNITE-6043:
------------------------------------------
I've misunderstood the implementation. That is, GridCacheSemaphoreImpl is
bounded and works similar to java.util.concurrent.Semaphore.
> Fix GridCacheSemaphoreImpl methods semantics
> --------------------------------------------
>
> Key: IGNITE-6043
> URL: https://issues.apache.org/jira/browse/IGNITE-6043
> Project: Ignite
> Issue Type: Bug
> Components: data structures
> Affects Versions: 2.1
> Reporter: Andrey Kuznetsov
> Assignee: Andrey Kuznetsov
> Priority: Critical
>
> Current semaphore implementation, {{GridCacheSemaphoreImpl}}, has inverted
> method semantics: {{acquire()}} releases the permit and {{release()}}
> acquires it. Also, debug-level method {{availablePermits()}} returns permits
> acquired so far. This confusing behaviour should be fixed.
> Also, it's worth noting in IgniteSemaphore's javadoc its unbounded nature, as
> opposed to {{java.util.concurrent.Semaphore}}.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
