Skip to content

fix: correctly handle nexttick scheduling in stream reads#24082

Merged
zcbenz merged 1 commit into9-x-yfrom
trop/9-x-y-bp-fix-correctly-handle-nexttick-scheduling-in-stream-reads-1591894586156
Jun 15, 2020
Merged

fix: correctly handle nexttick scheduling in stream reads#24082
zcbenz merged 1 commit into9-x-yfrom
trop/9-x-y-bp-fix-correctly-handle-nexttick-scheduling-in-stream-reads-1591894586156

Conversation

@trop
Copy link
Copy Markdown
Contributor

@trop trop Bot commented Jun 11, 2020

Backport of #24022

See that PR for details.

Notes: Fix an issue which would cause streaming protocol responses to stall in some cases

@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Jun 11, 2020
@trop trop Bot mentioned this pull request Jun 11, 2020
Merged
6 tasks
@trop trop Bot added 9-x-y backport This is a backport PR labels Jun 11, 2020
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Jun 11, 2020
@zcbenz zcbenz merged commit db7ac35 into 9-x-y Jun 15, 2020
@release-clerk
Copy link
Copy Markdown

release-clerk Bot commented Jun 15, 2020

Release Notes Persisted

Fix an issue which would cause streaming protocol responses to stall in some cases

@zcbenz zcbenz deleted the trop/9-x-y-bp-fix-correctly-handle-nexttick-scheduling-in-stream-reads-1591894586156 branch June 15, 2020 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@codebytere codebytere codebytere approved these changes

+1 more reviewer

@zcbenz zcbenz zcbenz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

9-x-y backport This is a backport PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zcbenz @codebytere @pfrazee