Skip to content

fix: add timeout to HTTP client in StarRocks stream load#8770

Closed
warren830 wants to merge 1 commit intomainfrom
fix/http-client-timeout
Closed

fix: add timeout to HTTP client in StarRocks stream load#8770
warren830 wants to merge 1 commit intomainfrom
fix/http-client-timeout

Conversation

@warren830
Copy link
Contributor

Summary

  • Add a 60-second timeout to the http.Client used in StarRocks stream load
  • Without a timeout, requests could hang indefinitely if the StarRocks server becomes unresponsive

Test plan

  • Verify StarRocks stream load still works correctly
  • Verify timeout triggers appropriately for unresponsive servers

The http.Client used for StarRocks stream load had no timeout configured,
which could cause requests to hang indefinitely. Add a 60-second timeout.
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. component/plugins This issue or PR relates to plugins improvement pr-type/bug-fix This PR fixes a bug labels Mar 16, 2026
@warren830 warren830 closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins improvement pr-type/bug-fix This PR fixes a bug size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant