-
Notifications
You must be signed in to change notification settings - Fork 281
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 LMDB Support #74
Add LMDB Support #74
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks for the PR!
We may need to start using public tensorflow api endpoints (vs. internal api from tensorflow.python....
) to make sure it works for future versions. Though this is an issue not solely related to LMDB, and could be done later.
Yes, I agree that we should switch to use public APIs. We can do it altogether with other dataset ops in separate PRs. |
This PR adds support for Lightning Memory-Mapped Database (LMDB). See #73 for background.