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

Prevent client from deadlocking/halting on large transfer by throwing error #61

Closed
wants to merge 1 commit into from

Conversation

KolbyML
Copy link
Member

@KolbyML KolbyML commented May 26, 2023

The issue can be read in this PR #56

This is a quick solution to the problem, with a further improvement added as a TODO

ping for request to review @jacobkaufmann

@KolbyML KolbyML changed the title Add else condition if user attempts to write over buffer capacity Prevent client from halting on large transfer by throwing error May 26, 2023
@KolbyML KolbyML force-pushed the return-error-add-todo branch from 0959ed0 to 4b549f4 Compare May 26, 2023 06:30
@KolbyML KolbyML changed the title Prevent client from halting on large transfer by throwing error Prevent client from deadlocking/halting on large transfer by throwing error May 28, 2023
@KolbyML
Copy link
Member Author

KolbyML commented May 29, 2023

Closing since I think we should merge #65 as the temporary solution to solve this problem. It adds complexity, but we will have to chunk data either way. Hopefully soon we can refactor the two pending write buffers into 1

@KolbyML KolbyML closed this May 29, 2023
@KolbyML KolbyML mentioned this pull request Jun 4, 2023
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