-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Change all messages to pass back their checksum. Switch to callback style #73015
Change all messages to pass back their checksum. Switch to callback style #73015
Conversation
… than what is in the asset cache
|
||
// We flush after each item as that forms a reasonably sized chunk of data to want to then send over the | ||
// pipe for the reader on the other side to read. This allows the item-writing to remain entirely | ||
// synchronous without any blocking on async flushing, while also ensuring that we're not buffering the | ||
// entire stream of data into the pipe before it gets sent to the other side. | ||
WriteAsset(writer, serializer, context, asset, cancellationToken); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
inliend this method.
|
||
await RequestAssetsAsync( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
inlined this method call.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jasonmalinowski For review when you get back. |
Extracted from #73013 to help keep that PR simpler.