-
Notifications
You must be signed in to change notification settings - Fork 101
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Describe the solution you'd like
We need to support client keep alive. Up for debate:
- Whether it remains off by default and opt-in or we turn it on by default w/ a fairly high value
- I'm leaning to on by default (EDIT: Seems it was decided to turn on by default in [Feature Request] SDK connections should enable keep-alive by default and allow keep-alive customization features#123)
- Whether to use TCP keep alive or HTTP/2 keep alive
- I'm leaning HTTP/2 keep alive, it's the higher level gRPC one that does a better job and is more configurable
- Which options to expose to users
- I'm leaning to interval, timeout, and while-idle
edziegle and StanislavPrusac
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request