Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Only request grpc write when not complete #10284

Closed
wants to merge 3 commits into from

Conversation

werkt
Copy link
Contributor

@werkt werkt commented Nov 21, 2019

If a queryWriteStatus yields a committedSize which leaves no content
remaining to be uploaded, immediately succeed a blob upload. This can
easily occur if a competing blob write completes asynchronously between
abnormal write termination and a query.

If a queryWriteStatus yields a committedSize which leaves no content
remaining to be uploaded, immediately succeed a blob upload. This can
easily occur if a competing blob write completes asynchronously between
abnormal write termination and a query.
@werkt werkt force-pushed the write-chunker-exhaustion branch from 18e2d0c to eff9081 Compare November 21, 2019 00:48
@buchgr
Copy link
Contributor

buchgr commented Nov 21, 2019

@werkt please add a test for this 😛

@bazel-io bazel-io closed this in cc2b3ec Nov 22, 2019
@werkt werkt deleted the write-chunker-exhaustion branch November 25, 2019 15:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants