[FEA] Remove merge strategy enum from merge APIs
Open
@benfred is already working on this.
Since Nov 20, 2025.
feature request
- Dominant language
- Cuda
- Stars
- 854
- Forks
- 236
- Avg merge
- 3d 3h
- Merged PRs (30d)
- 62
Description
The first iteration of the merge() APIs was going to include two options for merging- a logical and physical merge. We've since decided that it only makes sense to have first-class functions for physical merging and logical merging can be done via the construction (via factory function) of a composite_index (which we'll be refactoring later).
There are still some remnants of the merge strategy in the merge() apis and they should be removed altogether.
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.
Assessment
This issue has not been assessed yet.