-
Notifications
You must be signed in to change notification settings - Fork 138
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
Add a flag to control auto-deploy behavior #2276
Conversation
Signed-off-by: Sicheng Song <sicheng.song@outlook.com>
Signed-off-by: Sicheng Song <sicheng.song@outlook.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2276 +/- ##
============================================
- Coverage 81.42% 81.30% -0.12%
- Complexity 5853 5857 +4
============================================
Files 557 558 +1
Lines 23871 23952 +81
Branches 2493 2511 +18
============================================
+ Hits 19436 19475 +39
- Misses 3412 3441 +29
- Partials 1023 1036 +13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
common/src/main/java/org/opensearch/ml/common/model/MLDeploySetting.java
Show resolved
Hide resolved
Signed-off-by: Sicheng Song <sicheng.song@outlook.com>
* Add a flag to control auto-deploy behavior Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Fix compilation Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Bump back schema version due to already bumped in guardrail pr Signed-off-by: Sicheng Song <sicheng.song@outlook.com> --------- Signed-off-by: Sicheng Song <sicheng.song@outlook.com> (cherry picked from commit 22fcaf0)
* Add a flag to control auto-deploy behavior Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Fix compilation Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Bump back schema version due to already bumped in guardrail pr Signed-off-by: Sicheng Song <sicheng.song@outlook.com> --------- Signed-off-by: Sicheng Song <sicheng.song@outlook.com> (cherry picked from commit 22fcaf0)
* Add a flag to control auto-deploy behavior Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Fix compilation Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Bump back schema version due to already bumped in guardrail pr Signed-off-by: Sicheng Song <sicheng.song@outlook.com> --------- Signed-off-by: Sicheng Song <sicheng.song@outlook.com> (cherry picked from commit 22fcaf0) Co-authored-by: Sicheng Song <sicheng.song@outlook.com>
* Add a flag to control auto-deploy behavior Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Fix compilation Signed-off-by: Sicheng Song <sicheng.song@outlook.com> * Bump back schema version due to already bumped in guardrail pr Signed-off-by: Sicheng Song <sicheng.song@outlook.com> --------- Signed-off-by: Sicheng Song <sicheng.song@outlook.com> (cherry picked from commit 22fcaf0) Co-authored-by: Sicheng Song <sicheng.song@outlook.com>
Description
This PR introduced a boolean flag to control auto-deploy behavior. This boolean is wrapped with a new flat object field in order to avoid adding to much field to model index directly in the future. To make model index cleaner and maintainable.
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.