-
Notifications
You must be signed in to change notification settings - Fork 312
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
koord-scheduler: clarify framework extender #1066
koord-scheduler: clarify framework extender #1066
Conversation
a6ace53
to
873b308
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1066 +/- ##
==========================================
+ Coverage 67.25% 67.40% +0.15%
==========================================
Files 255 258 +3
Lines 28619 28684 +65
==========================================
+ Hits 19248 19335 +87
+ Misses 8031 8002 -29
- Partials 1340 1347 +7
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Joseph <joseph.t.lee@outlook.com>
873b308
to
6687964
Compare
The design document needs to be updated. Another picture saves a thousand words, draw a diagram of the internal architecture of the scheduler after expansion? |
Yeah.. Should update the design proposal..I want to update the proposal later when the enhanced implementation of Reservation is almost complete. If need, I also can update the proposal in the PR. WDYT? @hormes |
Or create an issue to track all the changes you mentioned, so that everyone will not miss it |
OKay. #1076 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Ⅰ. Describe what this PR does
The internal concepts of the
frameworkext
package of koord-scheduler are somewhat confusing and need some clarification.ExtenderFactory
is responsible for creatingExtender
.Extender
implementsExtendedHandle
, which means the extension offramework.Handle
, and implements some functions offramework.Framework
, responsible for intercepting some function calls. TheExtender
corresponds to each profile, and some transformers inside eachExtender
are initialized through the type assert mechanism when the plugin is built.Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist
make test