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

Gg/handle stream exception #431

Merged
merged 2 commits into from
Nov 27, 2024
Merged

Gg/handle stream exception #431

merged 2 commits into from
Nov 27, 2024

Conversation

genzgd
Copy link
Collaborator

@genzgd genzgd commented Nov 27, 2024

Summary

Treat low level HTTP chunk errors as "End of Stream" to invoke correct upstream error handling

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@genzgd genzgd merged commit b6a60a9 into main Nov 27, 2024
59 checks passed
@genzgd genzgd deleted the gg/handle_stream_exception branch November 27, 2024 20:08
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.

1 participant