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

Make keepalive interval configurable #838

Merged
merged 2 commits into from
Sep 2, 2022

Conversation

nodece
Copy link
Member

@nodece nodece commented Sep 2, 2022

Motivation

Fix #837

Modifications

  • Add the KeepAliveInterval in the ClientOptions

Signed-off-by: Zixuan Liu <nodeces@gmail.com>
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
@nodece nodece requested review from merlimat, cckellogg and wolfstudy and removed request for cckellogg September 2, 2022 16:08
@merlimat merlimat added this to the v0.9.0 milestone Sep 2, 2022
@merlimat merlimat merged commit f8dc88e into apache:master Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support keepalive as an optional parameter
2 participants