-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix multiple failing server tests. #453
Conversation
✅ DCO Check Passed 82e94f71671df45835cbe17c21dd5601f8b3e8d2 |
✅ Gradle Wrapper Validation success 82e94f71671df45835cbe17c21dd5601f8b3e8d2 |
✅ Gradle Precommit success 82e94f71671df45835cbe17c21dd5601f8b3e8d2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall changes look good to me.
Thanks for the change!
Dropped a single comment.
...r/src/test/java/org/opensearch/cluster/routing/allocation/BalanceUnbalancedClusterTests.java
Show resolved
Hide resolved
This commit fixes the failing unit tests in the server module which started failing as a result of the renaming. Signed-off-by: Rabi Panda <adnapibar@gmail.com>
✅ Gradle Wrapper Validation success dcbb8dd |
✅ DCO Check Passed dcbb8dd |
✅ Gradle Precommit success dcbb8dd |
This commit fixes some renaming issues which as a result fixes multiple failing unit tests in the server module.
Issue #441
Signed-off-by: Rabi Panda adnapibar@gmail.com