Remove NO_MORE_DOCS assertion as Lucene 10 APIs changed#17721
Remove NO_MORE_DOCS assertion as Lucene 10 APIs changed#17721expani wants to merge 5 commits intoopensearch-project:mainfrom
Conversation
Signed-off-by: expani <anijainc@amazon.com>
|
❌ Gradle check result for 4d6126e: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 7d02b34: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17721 +/- ##
============================================
- Coverage 72.55% 72.47% -0.08%
+ Complexity 65966 65901 -65
============================================
Files 5326 5326
Lines 305548 305548
Branches 44322 44322
============================================
- Hits 221684 221460 -224
- Misses 65759 65927 +168
- Partials 18105 18161 +56 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
msfroh
left a comment
There was a problem hiding this comment.
Can we replace these with assertions on values.docValueCount()?
Signed-off-by: expani <anijainc@amazon.com>
|
❕ Gradle check result for bc66a60: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
|
Thanks for reviewing @msfroh |
|
Closed in favor of #17649 |
Description
We found one failing test here with this draft PR from @msfroh to check which code paths don't honor SortedSetDocValues API constraints.
This fixes the tests affected by assertions on
NO_MORE_DOCSinAbstractStringFieldDataTestCaseafter all ords are iterated over. It was removed in Lucene 10 following deprecation using NO_MORE_ORDS