gaussianrecurrence opened a new pull request #699:
URL: https://github.com/apache/geode-native/pull/699
- Removed pool lock when registering a new DM.
- This solves the thread inter-lock whenever putting back connections
while recovering subscription on a failed endpoint.
---
**Context**: During our tests, whenever a server goes down, all the threads
in the geode-native client were getting stuck while trying to recover
subscriptions on the endpoint which is down. This was due to registerDM method
holding the pool lock.
As this pool was only used to guarantee the max-connections hard-limit and
it seems that's not a hard-limit anymore as @albertogpz demonstrated with PR
#676, then I'd say it's safe to remove this lock, avoiding therefore any
dead-locks.
----------------------------------------------------------------
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:
[email protected]
