-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
Remove option for unlimited parallelism #48218
Remove option for unlimited parallelism #48218
Conversation
Given the discussion in #41055 I think we better have mailing list discussion on this. WDYT? |
I'll drop a note on devlist. --> https://lists.apache.org/thread/jlroywp6tlcrgdsx28shl1nkll8bpvbt |
7207319
to
06ccb3a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lovely! Thanks for grabbing this one @jscheffl! I've had it on a ToDo sticky note on my laptop for couple days now :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. one question regarding newsfragment
No objections received. Merging. |
* Remove option for unlimited parallelism * Add newsfragment * Update newsfragments/48218.significant.rst Co-authored-by: Jed Cunningham <[email protected]> * Fix newsfragment static check --------- Co-authored-by: Jed Cunningham <[email protected]>
Last moment as I was looking through the open tasks to make some cleanup before Airflow 3.0
closes: #41162
related: #41055, #41107