-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
reuse Write and ReadOptions #8
Comments
Oh man, in case someone comes along and sees my poor (cold induced) word choice: s/immutable/unmodified by leveldb/ Nice job getting this in so fast. |
No problem, thanks for looking at our code and telling us how to make it On Tue, Nov 5, 2013 at 4:21 PM, Jeff Hodges notifications@github.comwrote:
|
It is (unless you modify them yourself, of course)! |
levigo.WriteOptions and levigo.ReadOptions are immutable config options that you can make once for each config that you need, and reuse. There's a few places in LevelDbDatastore that needlessly create them on every method call.
The text was updated successfully, but these errors were encountered: