[GEODE-7061] Under heavy load and many threads the C++ Native client may open lots of connections

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: native client
• Assignee: Alberto Gomez
• Reporter: Alberto Gomez
• Created: 08/Aug/19 10:28
• Updated: 30/Dec/19 18:51
• Resolved: 28/Aug/19 06:42

Description

Under heavy load and many threads (>30) the C++ Native client tends to open a lot of connections which, if the idleTimeout is not relatively small, could provoke client port exhaustion.

The reason for this uncontrolled creation of connections is the implementation of the thread managing connections (ThinClientPoolDM::cleanStaleConnections()) that, in order to determine which connections to close due to timeout or load conditioning, gets all the connections from the pool for some time. Threads requiring a connection when this operation is fired, will create new connections while the maximum is not reached.

The proposed solution consists of changing the implementation of the cleanStaleConnections so that it does not get all the connections from the pool at some point but instead, takes one at a time in order to determine if it should close it.

Issue Links

• links to: GitHub Pull Request #511