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

Update DocIterator #2060

Merged
merged 3 commits into from
Oct 17, 2024
Merged

Conversation

yangzq50
Copy link
Contributor

What problem does this PR solve?

Update DocIterator
Sort children of AndIterator by EstimateIterateCost instead of DocFreq

Issue link:#1862

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • Refactoring

@yangzq50 yangzq50 added the ci PR can be test label Oct 17, 2024
@yangzq50 yangzq50 force-pushed the update_minshouldmatch_iter branch from b8bd943 to a3f7584 Compare October 17, 2024 06:31
@JinHai-CN JinHai-CN merged commit ae34fad into infiniflow:main Oct 17, 2024
4 checks passed
@yangzq50 yangzq50 deleted the update_minshouldmatch_iter branch October 17, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci PR can be test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants