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.

Sign up for GitHub

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

Jump to bottom

Add keepalive in grpc channel for Bigtable client #181

Closed
HemangChothani opened this issue Dec 4, 2020 · 0 comments · Fixed by #182
Closed

Add keepalive in grpc channel for Bigtable client #181

HemangChothani opened this issue Dec 4, 2020 · 0 comments · Fixed by #182
Assignees
@HemangChothani
Labels
api: bigtable Issues related to the googleapis/python-bigtable API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@HemangChothani
Copy link
Contributor

Sets KeepAlive timer to 30s (Google Frontends are configured limits keepalive calls at 30s by default. If used below 30s, grpc automatically increases keepalive time by 2x after too_many_ping commands).

Sets KeepAliveTimeout to 10s (conservative)

Sets keepAliveWithoutCalls to false as it can potentially increase load.

Reference issue: googleapis/java-bigtable-hbase#2681
@product-auto-label product-auto-label bot added the api: bigtable Issues related to the googleapis/python-bigtable API. label Dec 4, 2020
@HemangChothani HemangChothani self-assigned this Dec 4, 2020
@HemangChothani HemangChothani added the type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. label Dec 4, 2020
@HemangChothani HemangChothani mentioned this issue Dec 4, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HemangChothani HemangChothani

Labels
api: bigtable Issues related to the googleapis/python-bigtable API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add keep alive timeout MaxxleLLC/python-bigtable
1 participant
@HemangChothani