apache / apache/lucene

extend TopGroups.merge test coverage [LUCENE-9010]

Open
#10,053 8 comments 0 reactions 0 assignees View on GitHub
legacy-jira-priority:Minor type:test
Dominant language
Java
Stars
3.6k
Forks
1.4k
Avg merge
2d 11h
Merged PRs (30d)
88

Description

This ~~sub~~-task proposes to add test coverage for the `TopGroups.merge` method, separately from but as preparation for #10039 fixing the 'maxScore is sometimes missing' bug.

edit: converted from #10039 bug-fix sub-task to test-type task of its own so that its life can extend beyond the #10039 bug-fix ticket's life for further test coverage extensions

---
Migrated from [LUCENE-9010](https://issues.apache.org/jira/browse/LUCENE-9010) by Christine Poerschke (@cpoerschke), updated Nov 12 2019
Attachments: [LUCENE-9010.patch](https://apache.github.io/lucene-jira-archive/attachments/LUCENE-9010/LUCENE-9010.patch) (versions: 2)
Linked issues:
- #10039
- #10084

Contributor guide

Open the contributing guide

Research direction

Start by locating the TopGroups.merge method and its existing tests in the Lucene repository, then review the linked #10039 and #10084 issues for relevant scenarios. Add coverage for the merge behavior described by the issue and run the affected test suite; done means the cases are captured independently of the maxScore bug fix.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
search
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.