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

81 supplier plugin health and store usage after revert #89

Conversation

JohannesDaniel
Copy link
Collaborator

@JohannesDaniel JohannesDaniel commented Dec 13, 2024

Description

This PR includes some refactorings for comments in #84

@JohannesDaniel
Copy link
Collaborator Author

@ylwu-amzn new PR after revert.

Fixed

  • code in comment in StoreUtils
  • var naming in PluginHealthStatusSupplier
  • license in PluginHealthStatusSupplier

@ylwu-amzn
Copy link

I added some comments to the old PR, can you help address those comments first ? #84
I can take another look once that comments addressed

@sstults sstults merged commit aced4bd into opensearch-project:main Dec 16, 2024
5 checks passed
sstults pushed a commit to sstults/opensearch-learning-to-rank-base that referenced this pull request Dec 19, 2024
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.

4 participants