Skip to content
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

Overlord can support many autoscalers of different categories #8989 #1

Merged
merged 23 commits into from
Feb 11, 2020

Conversation

sascha-coenen
Copy link

Fixes apache#8695.

Description

Added support of many autoscalers of different categories to serve tasks of according category.


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths.
  • added integration tests.
  • been tested in a test Druid cluster.

Key changed/added classes in this PR
  • CategoriedWorkerBehaviorConfig
  • PendingTaskBasedWorkerProvisioningStrategy
  • SimpleWorkerProvisioningStrategy

Vladimir Iordanov added 23 commits December 4, 2019 15:37
 - Changed behavior config structure.
 - Small refactoring
 - Extended unit tests
 - Added "category" field into Autoscaler
 - Changed config format to support the "category" field
 - Fixed parameters of unit tests
 - Put category information into Autoscaler
 - Changed structure of behavior config
 - Changed CategoriedProvisioningStrategy accordingly
 - Fixed bug in PendingTaskBasedWorkerProvisioningStrategy
 - Extended unit tests
 - Code cleanup and refactoring
 - Introduced new logic into the existing strategies
 - Added more test cases
 - Refactored unit tests
 - Used Default Worker Category as a fallback for null worker categories
 - Refactored legacy code to avoid NPE
 - Removed duplicates
 - Added more debug info
 - Removed duplicated code
@sascha-coenen sascha-coenen merged commit 359f73f into smaato:feature-8695 Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant