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

Improve set deletions percentage javadoc #12828

Merged
merged 5 commits into from
Jan 22, 2025

Conversation

yugushihuang
Copy link
Contributor

As a follow item of Discussion issue, improve the java doc to make the formula of deletion percentage clear to developers.

* Controls the maximum percentage of deleted documents that is tolerated in the index. Lower
* values make the index more space efficient at the expense of increased CPU and I/O activity.
* Values must be between 5 and 50. Default value is 20.
* Sets the maximum percentage of deleted documents that is tolerated in the index. The
Copy link
Contributor

@shubhamvishu shubhamvishu Nov 21, 2023

Choose a reason for hiding this comment

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

Maybe it'll be great if we include what @jpountz defined it on the issue here as well since it was easier to understand...."how much of my doc ID space is taken by deleted docs" -> "percentage of doc id space taken by deleted docs". WDYT?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the review, I will modify the wording.

Copy link

github-actions bot commented Jan 8, 2024

This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the [email protected] list. Thank you for your contribution!

@github-actions github-actions bot added the Stale label Jan 8, 2024
@jpountz
Copy link
Contributor

jpountz commented Feb 23, 2024

It looks like this PR is in a bad state somehow?

@github-actions github-actions bot removed the Stale label Feb 24, 2024
Copy link

github-actions bot commented Mar 9, 2024

This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the [email protected] list. Thank you for your contribution!

@github-actions github-actions bot added the Stale label Mar 9, 2024
@yugushihuang
Copy link
Contributor Author

@jpountz I have sync the fork. @shubhamvishu has already approved the commit. I will merge once the check passes.

@github-actions github-actions bot removed the Stale label Jan 11, 2025
@yugushihuang
Copy link
Contributor Author

@jpountz It looks like I do not have push access to the repo. Can you help me merge it. Thanks!

Copy link
Contributor

@msokolov msokolov left a comment

Choose a reason for hiding this comment

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

LGTM

@msokolov msokolov merged commit c8160b1 into apache:main Jan 22, 2025
5 checks passed
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.

4 participants