warrenfalk / rocksdb-sharp

.net bindings for the rocksdb by facebook
Other
206 stars 65 forks source link

MergeOperator proposal #70

Open pablofrommars opened 4 years ago

pablofrommars commented 4 years ago

Fix for #69

Proposed interface for user defined PartialMerge and FullMerge functions:

https://github.com/pablofrommars/rocksdb-sharp/blob/aa30a9fee098c92ade78532cad9a8bd16764fc3c/RocksDbSharp/MergeOperator.cs#L18-L40

The proposed implementation allocates buffers during marshalling. For performance reasons, the use of ArrayPool is suggested, happy to do the work in case you are interrested.