-
Notifications
You must be signed in to change notification settings - Fork 0
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
Kevel/lmdb 0.9.29 #1
base: master
Are you sure you want to change the base?
Conversation
git-subtree-dir: dependencies/lmdb git-subtree-split: 8ad7be2510414b9506ec9f9e24f24d04d9b04a1a
24149bc
to
a55c1c5
Compare
a55c1c5
to
b1aec5b
Compare
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.
This all looks good. I agree we will want to spend some more time with how we handle transactions but we can come back to that especially if we aren't getting burned by the cost to open a transaction.
Co-authored-by: Sam Hewitt <[email protected]>
@shooit Bifrost opens a single transaction per request and stores that in the AsyncLocalStorage context which the Store class looks for, so we're ok atm. |
No description provided.