Skip to content

Conversation

@jedcunningham
Copy link
Member

This rewords the description and changes the type of
worker_enable_remote_control and sets a default for
worker_prefetch_multiplier instead of relying on fallback.

@jedcunningham jedcunningham added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Feb 28, 2022
@jedcunningham jedcunningham marked this pull request as ready for review March 1, 2022 20:12
@jedcunningham
Copy link
Member Author

cc @pingzh

@pingzh
Copy link
Contributor

pingzh commented Mar 1, 2022

@jedcunningham good catch. thanks

@github-actions
Copy link

github-actions bot commented Mar 4, 2022

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Mar 4, 2022
This rewords the description and changes the type of
``worker_enable_remote_control`` and sets a default for
``worker_prefetch_multiplier`` instead of relying on fallback.
@jedcunningham jedcunningham merged commit f5f2216 into apache:main Mar 4, 2022
@jedcunningham jedcunningham deleted the celery_fixup branch March 4, 2022 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants