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

perf(bigtable): reduce latency #11032

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bhshkh
Copy link
Contributor

@bhshkh bhshkh commented Oct 24, 2024

Notice the timestamps in the logs.
I couldn't simulate a noticeable server side delay while sending trailers but there is microsecond delay can be observed.

# Below is printed when response is received by the end user
024-10-24 22:22:20.996658405 +0000 UTC m=+5.020035733  Successfully read row with key  row-00004

# Below is printed when client library records metrics
2024-10-24 22:22:20.996639015 +0000 UTC m=+5.020016332  recordAttemptCompletion Recording attempt_latencies

# Below is printed when client library records metrics
2024-10-24 22:22:20.996805456 +0000 UTC m=+5.020182782  recordAttemptCompletion Recording server_latencies

# Below is printed when client library records metrics
2024-10-24 22:22:20.996929846 +0000 UTC m=+5.020307163  recordOperationCompletion Recording operation_latencies

# Below is printed when client library records metrics
2024-10-24 22:22:20.997028566 +0000 UTC m=+5.020405893  recordOperationCompletion Recording retry_count

@product-auto-label product-auto-label bot added the api: bigtable Issues related to the Bigtable API. label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the Bigtable API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant