-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update service db to use store #1244
Conversation
…base system that accept a store.
This pull request has been mentioned on MESG Community. There might be relevant details there: https://forum.mesg.com/t/network-implementation-with-cosmossdk/304/15 |
@@ -82,7 +84,10 @@ func TestServiceDBDelete(t *testing.T) { | |||
require.IsType(t, &ErrNotFound{}, err) | |||
} | |||
|
|||
// TOFIX: the database is not thread safe anymore... |
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.
the database is still thread safe, but the question is why the transaction has gone
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.
kvstore does not implement transaction as it should modify and read the store one transaction at a time. so they don't need concurrent access..
# Conflicts: # database/service_db.go
Dependant on #1243