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
[FLINK-21390] Rename DeclarativeScheduler to AdaptiveScheduler #14970
Conversation
…tPoolService and Scheduler factories The SlotPoolServiceSchedulerFactory bundles the SlotPoolServiceFactory and SchedulerNGFactory to make sure that we only instantiate valid SlotPoolService and SchedulerNG combinations.
Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community Automated ChecksLast check on commit c12ff62 (Fri Feb 19 17:34:46 UTC 2021) Warnings:
Mention the bot in a comment to re-run the automated checks. Review Progress
Please see the Pull Request Review Guide for a full explanation of the review process. The Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required Bot commandsThe @flinkbot bot supports the following commands:
|
…dle SlotPoolService and Scheduler factories
Note that this commit also changes the system property to enable the adaptive scheduler from flink.tests.enable-declarative-scheduler to flink.tests.enable-adaptive-scheduler.
c12ff62
to
08cce5c
Compare
Thanks for the review @zentol. Merging this PR now. |
Why were the configuration keys not renamed in this PR? |
I forgot to rename them. I will update them in a hotfix commit. |
Thanks a lot! |
Renames the
DeclarativeScheduler
intoAdaptiveScheduler
.Note that this commit also changes the system property to enable the adaptive scheduler
from
flink.tests.enable-declarative-scheduler
toflink.tests.enable-adaptive-scheduler
.This PR is based on #14969.