Skip to content

Conversation

@ylwu-amzn
Copy link
Collaborator

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

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.

b4sjoo and others added 2 commits March 2, 2023 14:04
Signed-off-by: Sicheng Song <sicheng.song@outlook.com>

Signed-off-by: Sicheng Song <sicheng.song@outlook.com>
Signed-off-by: Yaliang Wu <ylwu@amazon.com>

Signed-off-by: Yaliang Wu <ylwu@amazon.com>
@ylwu-amzn ylwu-amzn requested a review from a team March 2, 2023 22:40
@opensearch-trigger-bot opensearch-trigger-bot bot added the documentation Improvements or additions to documentation label Mar 2, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #780 (29df28c) into main (56cdb85) will decrease coverage by 0.33%.
The diff coverage is 0.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main     #780      +/-   ##
============================================
- Coverage     84.17%   83.85%   -0.33%     
+ Complexity     1039     1035       -4     
============================================
  Files            99       99              
  Lines          3740     3740              
  Branches        344      344              
============================================
- Hits           3148     3136      -12     
- Misses          445      456      +11     
- Partials        147      148       +1     
Flag Coverage Δ
ml-commons 83.85% <0.00%> (-0.33%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../java/org/opensearch/ml/task/MLTaskDispatcher.java 32.94% <0.00%> (ø)
.../cluster/MLCommonsClusterManagerEventListener.java 65.62% <0.00%> (-12.50%) ⬇️
...ain/java/org/opensearch/ml/task/MLTaskManager.java 61.92% <0.00%> (-4.07%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ylwu-amzn ylwu-amzn merged commit a4c5c84 into opensearch-project:main Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants