Skip to content

Conversation

@bcherry
Copy link
Contributor

@bcherry bcherry commented Oct 8, 2025

There are some cases where we see ~6s delays. We are separately working to track those down and improve the worst-case speed. The DC reliability work from the summer has addressed the vast majority of true lost packets so it's better to keep waiting than to fail early.

Three changes

  • Increase max RT to 7s (it needs to be at least 6s, this felt like a conservative choice for now)
  • Increase default response timeout by 15s (keeps the effective handler time to 8s as it was already)
  • Add missing response timeout adjustment to account for RT time (other SDKs had this already)

@bcherry bcherry requested review from boks1971 and lukasIO October 8, 2025 19:00
Copy link
Contributor

@xianshijing-lk xianshijing-lk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two quick questions. lgtm

@xianshijing-lk
Copy link
Contributor

thanks, lgtm.

@bcherry bcherry merged commit ac14f09 into main Oct 9, 2025
19 checks passed
@bcherry bcherry deleted the bcherry/increase_rpc_rt branch October 9, 2025 21:54
@github-actions github-actions bot mentioned this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants