Skip to content

Why expired connections in thin pool are force closed? #393

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
golubovai opened this issue Sep 4, 2024 · 4 comments
Open

Why expired connections in thin pool are force closed? #393

golubovai opened this issue Sep 4, 2024 · 4 comments
Labels
inactive This was marked by stalebot as inactive question Further information is requested

Comments

@golubovai
Copy link

Hello! Force close of expired connections from pool prevent normal completion and execution of logoff triggers. Why are they force closed instead of normal close?

@golubovai golubovai added the question Further information is requested label Sep 4, 2024
@anthony-tuininga
Copy link
Member

The main reason is that simply disconnecting the socket is guaranteed whereas sending the LogOff request may result in a hang -- which could impact the pool negatively. What do you mean by "expired connections"? When using a pool the general concept is that which connection is used is immaterial and the pool should be able to substitute connections or rebuild them at will.

@golubovai
Copy link
Author

"expired connections" are idle connections in pool closed after timeout, set by so named parameter in pool constructor.
Our database tracks connections by means of logon/logoff triggers. These connections look like they were killed.

@cjbj
Copy link
Member

cjbj commented Nov 6, 2024

@golubovai what exactly does your app rely on the logoff trigger to do? From the app's point of view, the connection was no longer needed, or usable, when the app released it back to the pool. What do you gain from some, unknown time later running a trigger?

@anthony-tuininga we should revisit Thin/Thick behavior with the OCI team.

Copy link

stale bot commented Apr 26, 2025

This issue has been automatically marked as inactive because it has not been updated recently. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the inactive This was marked by stalebot as inactive label Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive This was marked by stalebot as inactive question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants