Skip to content
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

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

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

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

golubovai opened this issue Sep 4, 2024 · 3 comments
Labels
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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants