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

[fix](index compaction)Skip writing terms with a doc frequency of 0 #248

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

qidaye
Copy link
Contributor

@qidaye qidaye commented Oct 29, 2024

If the doc frequency of a term is 0, it means the term is deleted. So we should not write it.

Copy link
Member

@airborne12 airborne12 left a comment

Choose a reason for hiding this comment

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

Need test case

Copy link
Collaborator

@zzzxl1993 zzzxl1993 left a comment

Choose a reason for hiding this comment

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

LGTM

@airborne12 airborne12 merged commit 7cf6cf4 into apache:clucene Oct 31, 2024
3 of 4 checks passed
@qidaye qidaye deleted the fix_no_doc_term branch October 31, 2024 09:18
qidaye added a commit to qidaye/doris-thirdparty that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants