quickwit-oss / quickwit-oss/quickwit
Merge policy update footgun
Open
Nobody has claimed this yet.
- #4868 by @rdettai — closed without merging
enhancement
- Dominant language
- Rust
- Stars
- 11.7k
- Forks
- 597
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 37
Description
We are about to add the ability to update index configs, including their merge policies. You could start a "merge storm" by increasing the maturation period or the merge factor. Do we want to add a parameter to apply merge policy updates only to new splits.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review the issue's proposed merge-policy update behavior and the linked pull request #4868 first. Determine whether updates should apply only to new splits, then document the agreed behavior and identify the affected implementation and tests before starting work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- search
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100