facebook / facebook/rocksdb

Make MergeOperator's PartialMerge override optional with AllowSingleOperand

Open
#3,845 0 comments 0 reactions 0 assignees View on GitHub
enhancement up-for-grabs
Dominant language
C++
Stars
32.1k
Forks
6.9k
Avg merge
32m
Merged PRs (30d)
1

Description

Setting AllowSingleOperand to true requires users's mandatorily implementing PartialMerge/ParialMergeMulti as of today. This should be made optional in the spirit of how MergeOperator is implemented originally.

@sagar0 to add more context if necessary

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.