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.

Sign up for GitHub

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

Jump to bottom

gh-106527: asyncio: optimize to add/remove readers and writers #106528

Merged
merged 8 commits into from
Jul 22, 2023
Merged

gh-106527: asyncio: optimize to add/remove readers and writers #106528

merged 8 commits into from
Jul 22, 2023

Conversation

bdraco
Copy link
Contributor

@bdraco bdraco commented Jul 7, 2023 •
edited

Reduce overhead to add and remove asyncio readers and writers

@bedevere-bot bedevere-bot mentioned this pull request Jul 7, 2023
Closed
@bdraco bdraco marked this pull request as ready for review July 7, 2023 19:37
@bdraco bdraco mentioned this pull request Jul 12, 2023
Merged
@methane methane changed the title gh-106527: Reduce overhead to add and remove asyncio readers and writers gh-106527: asyncio: optimize adding and removing readers and writers Jul 22, 2023
@methane methane changed the title gh-106527: asyncio: optimize adding and removing readers and writers gh-106527: asyncio: optimize to add/remove readers and writers Jul 22, 2023
@methane methane enabled auto-merge (squash) July 22, 2023 16:35
@bdraco
Copy link
Contributor Author

bdraco commented Jul 22, 2023

windows x86_64 runner failed on an unrelated test test_concurrent_futures

@methane methane merged commit b7dc795 into python:main Jul 22, 2023
21 checks passed
@bdraco
Copy link
Contributor Author

bdraco commented Jul 22, 2023

thanks

@bdraco bdraco deleted the optimize_adding_removing_asyncio_writers branch July 22, 2023 21:21
bdraco added a commit to bdraco/docker-base that referenced this pull request Jul 22, 2023
@bdraco
Backport selector performance improvements from cpython
3ee9501 
python/cpython#106665
python/cpython#106555
python/cpython#106754
python/cpython#106528
@bdraco bdraco mentioned this pull request Jul 22, 2023
Merged
jtcave pushed a commit to jtcave/cpython that referenced this pull request Jul 23, 2023
@bdraco @jtcave
pythongh-106527: asyncio: optimize to add/remove readers and writers (p…
636f995 
…ython#106528)
mementum pushed a commit to mementum/cpython that referenced this pull request Jul 23, 2023
@bdraco @mementum
pythongh-106527: asyncio: optimize to add/remove readers and writers (p…
8259583 
…ython#106528)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@1st1 1st1 Awaiting requested review from 1st1 1st1 is a code owner

@asvetlov asvetlov Awaiting requested review from asvetlov asvetlov is a code owner

@gvanrossum gvanrossum Awaiting requested review from gvanrossum gvanrossum is a code owner

@kumaraditya303 kumaraditya303 Awaiting requested review from kumaraditya303 kumaraditya303 is a code owner

@willingc willingc Awaiting requested review from willingc willingc is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@bdraco @methane @bedevere-bot