Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is stream column family useful to be a seperate cf instead of merging into subkey cf? #2330

Open
1 of 2 tasks
PragmaTwice opened this issue May 25, 2024 · 4 comments
Open
1 of 2 tasks
Labels
enhancement type enhancement

Comments

@PragmaTwice
Copy link
Member

PragmaTwice commented May 25, 2024

Search before asking

  • I had searched in the issues and found no similar issues.

Motivation

As title, should we merge it to subkey cf while keeping compatible to old versions?

Solution

  • check the current cfs using ListColumnFamilies() before opening db:
    • if it's a new database or no stream cf:
      • don't create stream cf, use default as stream cf
    • if there's stream cf:
      • keep to use stream cf for compatibility

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@PragmaTwice PragmaTwice added the enhancement type enhancement label May 25, 2024
@PragmaTwice PragmaTwice changed the title Is stream column family useful to be an seperate cf instead of merging to subkey cf? May 25, 2024
@PragmaTwice PragmaTwice changed the title Is stream column family useful to be an seperate cf instead of merging into subkey cf? May 26, 2024
@git-hulk
Copy link
Member

@torwig @Yangsx-1 What do you think about this proposal?

@torwig
Copy link
Contributor

torwig commented May 27, 2024

If the only reason is compatibility, sure, we can do what @PragmaTwice offered.

@Yangsx-1
Copy link
Contributor

I think the amplification of stream is not very large, it's ok to merge it to subkey cf.

@git-hulk
Copy link
Member

I think the amplification of stream is not very large, it's ok to merge it to subkey cf.

Good point. It might affect the reading performance if there are too many tombstones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement type enhancement
4 participants