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

LUCENE-9449 Skip docs with _doc sort and "after" (#1725) #1856

Merged

Conversation

mayya-sharipova
Copy link
Contributor

  • Enhance DocComparator to provide an iterator over competitive
    documents when searching with "after". This iterator can quickly position
    on the desired "after" document skipping all documents and segments before
    "after".

  • Redesign numeric comparators to provide skipping functionality
    by default.

Backport for #LUCENE-9449

- Enhance DocComparator to provide an iterator over competitive
documents when searching with "after". This iterator can quickly position
on the desired "after" document skipping all documents and segments before
"after".

- Redesign numeric comparators to provide skipping functionality
by default.

Backport for #LUCENE-9449
@mayya-sharipova
Copy link
Contributor Author

@jimczi This is a backport for #1725. Not sure if you would like to review it, or I can just merge it.

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, I left one comment regarding the description in CHANGES.txt.

lucene/CHANGES.txt Outdated Show resolved Hide resolved
@mayya-sharipova mayya-sharipova merged commit 7542168 into apache:branch_8x Sep 17, 2020
@mayya-sharipova mayya-sharipova deleted the sort-by-doc-optim_8x branch September 17, 2020 15:46
mayya-sharipova added a commit that referenced this pull request Oct 7, 2020
Sort optimization on _doc was introduced in PR #1856,
but it looks unstable and lead to some recent tests failure.
As the release of 8.7 is very soon, we need to temporarily
disable this sort optimization for _doc for this release
with a plan to stabilize it for later releases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants