Skip to content

GH-74116: allow multiple drain waiters for asyncio.StreamWriter#94705

Merged
gvanrossum merged 3 commits into
python:mainfrom
kumaraditya303:multiple-drainers
Aug 29, 2022
Merged

GH-74116: allow multiple drain waiters for asyncio.StreamWriter#94705
gvanrossum merged 3 commits into
python:mainfrom
kumaraditya303:multiple-drainers

Conversation

@kumaraditya303

@kumaraditya303 kumaraditya303 commented Jul 9, 2022

Copy link
Copy Markdown
Contributor

Comment thread Lib/asyncio/streams.py Outdated
@kumaraditya303 kumaraditya303 force-pushed the multiple-drainers branch 2 times, most recently from efcfd1d to f6ede6e Compare August 1, 2022 13:54
@kumaraditya303 kumaraditya303 requested a review from ambv August 2, 2022 12:36
@kumaraditya303 kumaraditya303 requested review from gvanrossum and removed request for ambv and vstinner August 28, 2022 13:42

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay, I have a nit or two.

Comment thread Misc/NEWS.d/next/Library/2022-07-09-08-55-04.gh-issue-74116.0XwYC1.rst Outdated
Comment thread Lib/asyncio/streams.py Outdated
Comment thread Lib/asyncio/streams.py
Comment thread Lib/asyncio/streams.py

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG! I'll merge next.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @kumaraditya303 for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 29, 2022
…pythonGH-94705)

(cherry picked from commit e5b2453)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@bedevere-bot

Copy link
Copy Markdown

GH-96395 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Aug 29, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 29, 2022
…pythonGH-94705)

(cherry picked from commit e5b2453)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Aug 29, 2022
@bedevere-bot

Copy link
Copy Markdown

GH-96396 is a backport of this pull request to the 3.10 branch.

@kumaraditya303 kumaraditya303 deleted the multiple-drainers branch August 30, 2022 06:56
pablogsal pushed a commit that referenced this pull request Aug 30, 2022
…4705) (#96395)

(cherry picked from commit e5b2453)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Sep 8, 2022
…4705)

(cherry picked from commit e5b2453)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants