add support for system tcp keepalive options #262
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Currently the connector doesn't allow users to configure detailed keepalive settings on the socket requiring them to modify their system or session settings. This can be a challenge as many users are unfamiliar with these settings and even if they are they may not have access to those system settings or those settings can impact connecting to other applications.
Please note this PR is a rough proposal, likely needs some refinement and has not been properly tested. Happy to do more here if we're aligned on this approach.
Motivation and Context
See issue: #252 (comment)
Testing
TBD, more to come
Screenshots (if appropriate)
Types of changes
Checklist
./build.sh
succeedspytest test/unit
and they are passing.