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

Vladisav Jelisavcic commented on IGNITE-4173:
---------------------------------------------

This was a bug, the cause was that attempt to recover from failure was throwing 
the NPE for the newly created semaphores (because semaphores are lazily 
initialized).
There is also a problem with the  {{SemaphoreFailoverSafeReleasePermitsTest}}: 
second semaphore was created only after the first node failed; in this scenario 
there are no semaphores to attempt recovery so the permit is lost for good. 

> IgniteSemaphore with failoverSafe enabled doesn't release permits in case 
> permits owner node left topology
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-4173
>                 URL: https://issues.apache.org/jira/browse/IGNITE-4173
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 1.7
>            Reporter: Andrey Gura
>            Assignee: Vladisav Jelisavcic
>             Fix For: 2.0
>
>
> {{IgniteSemaphore}} with {{failoverSafe}} enabled doesn't release permits in 
> case permits owner node left topology.
> See reproducer in test class {{SemaphoreFailoverSafeReleasePermitsTest}}.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to