Skip to content

add rocksdb_sstfilewriter_create_with_column_family_handle#176

Open
ltagliamonte-dd wants to merge 2 commits intolinxGnu:masterfrom
ltagliamonte-dd:support_cf_handle
Open

add rocksdb_sstfilewriter_create_with_column_family_handle#176
ltagliamonte-dd wants to merge 2 commits intolinxGnu:masterfrom
ltagliamonte-dd:support_cf_handle

Conversation

@ltagliamonte-dd
Copy link

@ltagliamonte-dd ltagliamonte-dd commented Apr 3, 2025

allow to create a sstfilewriter with CF handler to pass CF information
@linxGnu please take a look

c.h
const rocksdb_envoptions_t* env, const rocksdb_options_t* io_options,
const rocksdb_comparator_t* comparator);
extern ROCKSDB_LIBRARY_API rocksdb_sstfilewriter_t*
rocksdb_sstfilewriter_create_with_column_family_handle(
Copy link
Owner

Choose a reason for hiding this comment

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

Hi @ltagliamonte-dd
I think you should drive your PR to upstream https://github.com/facebook/rocksdb/blob/v10.0.1/include/rocksdb/c.h instead

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.

2 participants