◐ Shell
reader mode source ↗
Skip to content

bpo-29842: Make Executor.map less eager so it handles large/unbounded…#18566

Closed
graingert wants to merge 18 commits into
python:mainfrom
graingert:fix-issue-29842
Closed

bpo-29842: Make Executor.map less eager so it handles large/unbounded…#18566
graingert wants to merge 18 commits into
python:mainfrom
graingert:fix-issue-29842

Conversation

@graingert

@graingert graingert commented Feb 20, 2020 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

… input iterables appropriately

bugs.python.org/issue29842

bugs.python.org/issue29842

recreate of #707 with conflicts fixed and versionchanged updated to py3.9

/cc @MojoVampire

https://bugs.python.org/issue29842

rdarder
rdarder reviewed Mar 22, 2021
View reviewed changes

@rdarder rdarder left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hide comment

Hi there! sorry for intruding. I was looking for this feature and decided to try this patch out. Works great! Just found a small issue so I sent you a comment.
Thank you!

@Tronic

Tronic commented Oct 29, 2021

Copy link
Copy Markdown

While waiting for this to be included in Python, one can easily copy&paste the map function out of this version and call it with the executor as the first argument. Hoping that you get this merged soon!

@MojoVampire MojoVampire added performance stdlib Standard Library Python modules in the Lib/ directory labels May 4, 2022
@MojoVampire MojoVampire mannequin mentioned this pull request Apr 10, 2022
Closed
@kumaraditya303 kumaraditya303 mentioned this pull request Jul 15, 2022
Closed
@kumaraditya303 kumaraditya303 self-requested a review July 18, 2022 15:30
@kumaraditya303 kumaraditya303 added type-feature A feature request or enhancement 3.12 only security fixes labels Jul 18, 2022
@kumaraditya303

kumaraditya303 commented Jul 23, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

@graingert Tests are failing, can you fix them and rebase to main? I'll then review and run through the buildbots.

@kumaraditya303

kumaraditya303 commented Jul 23, 2022

Copy link
Copy Markdown
Contributor

@graingert You can run make patchcheck to fix the docs CI.

@SvenDS9 SvenDS9 mentioned this pull request Feb 24, 2023
Closed
@Jason-Y-Z Jason-Y-Z mentioned this pull request Feb 3, 2024
Closed
@Jason-Y-Z

Jason-Y-Z commented Feb 3, 2024

Copy link
Copy Markdown
Contributor

Hey all, I took the inspiration from this PR and continued the work in #114975

@ebonnal ebonnal mentioned this pull request Oct 17, 2024
Merged
@ebonnal

ebonnal commented Oct 17, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hi, fyi here is a follow up PR: #125663 🙏🏻

@graingert graingert closed this Dec 28, 2024
@graingert graingert deleted the fix-issue-29842 branch December 28, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kumaraditya303 kumaraditya303 kumaraditya303 left review comments

+1 more reviewer

@rdarder rdarder rdarder left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

3.12 awaiting review performance Performance or resource usage stdlib Standard Library Python modules in the Lib/ directory type-feature A feature request or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@graingert @Tronic @kumaraditya303 @Jason-Y-Z @ebonnal @rdarder @the-knights-who-say-ni @ezio-melotti @MojoVampire @bedevere-bot