-
Notifications
You must be signed in to change notification settings - Fork 158
feat: Removes storage implementations (Couchdb, MySQL) #2245
feat: Removes storage implementations (Couchdb, MySQL) #2245
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2245 +/- ##
==========================================
+ Coverage 89.57% 89.81% +0.24%
==========================================
Files 221 219 -2
Lines 15094 14799 -295
==========================================
- Hits 13520 13292 -228
+ Misses 935 896 -39
+ Partials 639 611 -28
Continue to review full report at Codecov.
|
@@ -37,20 +36,20 @@ type StorePrefixWrapper struct { | |||
|
|||
// Put stores v with k ID by prefixing it with IDPrefix. | |||
func (b *StorePrefixWrapper) Put(k string, v []byte) error { | |||
if k == "" { | |||
return fmt.Errorf("cannot Put with empty key") | |||
if k != "" { |
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.
@Baha-sk @fqutishat
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.
We should not check it here. We need to rely on DB.
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.
I agree
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.
sure, it's ok then
Note: These are available as part of aries-framework-go-ext - https://github.com/hyperledger/aries-framework-go-ext/tree/main/component/storage |
Signed-off-by: Andrii Soluk <isoluchok@gmail.com>
This PR removes Couchdb and MySQL and implementations.
Signed-off-by: Andrii Soluk isoluchok@gmail.com