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

Add support for open_for_read_only APIs #402

Merged
merged 2 commits into from Apr 21, 2020

Conversation

wqfish
Copy link
Contributor

@wqfish wqfish commented Apr 9, 2020

This change exposes the C APIs that open RocksDB for read only.

src/db.rs Outdated Show resolved Hide resolved
src/db.rs Outdated Show resolved Hide resolved
src/db.rs Outdated Show resolved Hide resolved
src/db.rs Outdated Show resolved Hide resolved
@calavera
Copy link
Contributor

calavera commented Apr 20, 2020

@aleksuss, @vitvakatu I have implemented OpenAsSecondary on top of the AccessType enum introduced in this pull request. I was wonder when you're going to merge this one, so I can open a new pull request without complicating the review more than necessary.

You can see my changes in calavera@de8716e

@aleksuss aleksuss merged commit a8b77cf into rust-rocksdb:master Apr 21, 2020
1 check passed
@aleksuss
Copy link
Member

aleksuss commented Apr 21, 2020

@calavera I'm waiting for your PR. I think to include it in the 0.14.0 release.

@wqfish wqfish deleted the read_only branch Apr 22, 2020
rleungx pushed a commit to rleungx/rust-rocksdb that referenced this issue Jun 17, 2020
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.

None yet

3 participants