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

The client library does not set options SO_KEEPALIVE, TCP_NODELAY for socket at connection [CORE1763] #2187

Closed
firebird-automations opened this issue Feb 26, 2008 · 9 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: John Kilin (johnkilin)

Relate to CORE3387

The client library does not set options SO_KEEPALIVE, TCP_NODELAY for socket at connection

Commits: 5cb43d8

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @AlexPeshkoff

SO_KEEPALIVE is needed for server to watch lost clients. How do you imagine client watching for lost server?
What about TCP_NODELAY... I'm not sure, does this option missing at client breaks something in connectivity algorithm. Can you provide a sample? If needed, I can turn it on easily.

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

assignee: Alexander Peshkov [ alexpeshkoff ] => Vlad Khorsun [ hvlad ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

Client socket for database connection now set TCP_NODELAY option (accordingly to firebird.conf setting, by default on).
Client socket for evens processing now set SO_KEEPALIVE option

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 2.5 Alpha 1 [ 10224 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

Link: This issue relate to CORE3387 [ CORE3387 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

QA Status: No test

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Closed [ 6 ] => Closed [ 6 ]

QA Status: No test => Cannot be tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants