Skip to content

gh-88321: Add backlog arg for multiprocessing manager server #26332

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

junnplus
Copy link

@junnplus junnplus commented May 24, 2021

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Jun 24, 2021
@junnplus
Copy link
Author

junnplus commented Jan 7, 2022

It seems that a long time has passed, can someone help to review it?

Copy link
Contributor

@MaxwellDupre MaxwellDupre left a comment

Choose a reason for hiding this comment

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

Looks ok.

@mananshah99
Copy link

Curious about the state of this PR, ran into a similar issue recently.

@arhadthedev arhadthedev changed the title bpo-44155: Add backlog arg for multiprocessing manager server gh-88321: Add backlog arg for multiprocessing manager server Apr 26, 2023
@athompson673
Copy link

This may become a bigger problem as cpu's get more and more cores. I can't really think of a more complete solution than informing users via the documentation that ConnectionRefused may occur if the backlog fills up, and allowing them to increase it (to within the system limit).
I encountered this question on SO: https://stackoverflow.com/q/77042910/3220135 where the user solved the problem by polling the client side until the connection was not refused which worked for them, but I don't think is a general solution, and still requires editing the multiprocessing library.

@don-charlie-pc
Copy link

Hi! I hope this change gets in soon, as I'm running in a high end system and facing this issue a lot.

@ygutgutia
Copy link

Any updates on this? Can this be taken up? @junnplus

@junnplus junnplus force-pushed the bpo44155-listener-backlog-fix branch from 73d42d3 to 75d8404 Compare July 22, 2024 02:09
@junnplus junnplus requested a review from gpshead as a code owner July 22, 2024 02:09
@junnplus junnplus force-pushed the bpo44155-listener-backlog-fix branch from 75d8404 to 7f46f18 Compare July 22, 2024 02:14
@bedevere-app
Copy link

bedevere-app bot commented Jul 22, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@junnplus junnplus force-pushed the bpo44155-listener-backlog-fix branch from 769a3d9 to 7f46f18 Compare July 22, 2024 02:18
@bedevere-app
Copy link

bedevere-app bot commented Jul 22, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

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.

10 participants