You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2020/07/07 15:48:12 UTC

[GitHub] [ignite] ptupitsyn opened a new pull request #8003: IGNITE-13226 .NET: Fix ClientNotificationHandler leak in ClientSocket

ptupitsyn opened a new pull request #8003:
URL: https://github.com/apache/ignite/pull/8003


   Fix race condition in `ClientSocket.AddNotificationHandler`: `SetHandler` call was causing `RemoveNotificationHandler` call before `ConcurrentDictionary.AddOrUpdate` has finished, so `AddOrUpdate` added the handler back, causing the leak.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [ignite] ptupitsyn merged pull request #8003: IGNITE-13226 .NET: Fix ClientNotificationHandler leak in ClientSocket

Posted by GitBox <gi...@apache.org>.

ptupitsyn merged pull request #8003:
URL: https://github.com/apache/ignite/pull/8003


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org