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

Fix database open with column family. #12167

Closed
wants to merge 2 commits into from

Conversation

rhubner
Copy link
Contributor

@rhubner rhubner commented Dec 20, 2023

When is RocksDB is opened with Column Family descriptors, the default column family must be set properly. If it was not, then the flush operation will fail.

Copy link
Collaborator

@adamretter adamretter left a comment

Choose a reason for hiding this comment

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

In general this looks great thanks @rhubner, just two small changes.

java/src/main/java/org/rocksdb/RocksDB.java Outdated Show resolved Hide resolved
java/src/main/java/org/rocksdb/RocksDB.java Outdated Show resolved Hide resolved
@rhubner rhubner force-pushed the eb/column-family-fix branch from da97f8e to 49ed9a0 Compare December 21, 2023 12:35
@rhubner rhubner force-pushed the eb/column-family-fix branch from 49ed9a0 to 090761d Compare January 10, 2024 11:40
Copy link
Collaborator

@adamretter adamretter left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @rhubner

@adamretter adamretter marked this pull request as ready for review January 23, 2024 21:44
When is RocksDB open with column family descriptors, the default column
family must be set properly. If not, then the flush operation will fail.
@rhubner rhubner force-pushed the eb/column-family-fix branch from 090761d to b3fe98d Compare January 24, 2024 06:50
@facebook-github-bot
Copy link
Contributor

@cbi42 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@cbi42 merged this pull request in f2ddb92.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants