-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Cleanup unused code in org.opensearch.common.lucene #5306
Cleanup unused code in org.opensearch.common.lucene #5306
Conversation
Signed-off-by: Rabi Panda <adnapibar@gmail.com>
|
||
import static org.hamcrest.Matchers.equalTo; | ||
|
||
public class LuceneTests extends OpenSearchTestCase { | ||
private static final NamedWriteableRegistry EMPTY_REGISTRY = new NamedWriteableRegistry(Collections.emptyList()); | ||
|
||
public void testWaitForIndex() throws Exception { |
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.
Why this test is removed?
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.
The test is for Lucene.waitForIndex()
which is not used anywhere.
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.
Fair enough, thank you
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #5306 +/- ##
============================================
+ Coverage 70.98% 71.10% +0.11%
- Complexity 58121 58167 +46
============================================
Files 4704 4704
Lines 277312 277266 -46
Branches 40149 40147 -2
============================================
+ Hits 196854 197145 +291
+ Misses 64356 63975 -381
- Partials 16102 16146 +44
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Description
Cleanup unused internal code in the
o.o.common.lucene
packageIssues Resolved
N/A
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.
Signed-off-by: Rabi Panda adnapibar@gmail.com