refactor(store/v2): replace dboptions by dynamic config (#22185)

This commit is contained in:
Julien Robert 2024-10-10 12:25:26 +02:00 committed by GitHub
parent 04c1590a6e
commit 3a03804c14
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
6 changed files with 9 additions and 19 deletions

View File

@ -3,8 +3,8 @@ package db
import (
"fmt"
coreserver "cosmossdk.io/core/server"
corestore "cosmossdk.io/core/store"
"cosmossdk.io/store/v2"
)
type DBType string
@ -18,7 +18,7 @@ const (
DBFileSuffix string = ".db"
)
func NewDB(dbType DBType, name, dataDir string, opts store.DBOptions) (corestore.KVStoreWithBatch, error) {
func NewDB(dbType DBType, name, dataDir string, opts coreserver.DynamicConfig) (corestore.KVStoreWithBatch, error) {
switch dbType {
case DBTypeGoLevelDB:
return NewGoLevelDB(name, dataDir, opts)

View File

@ -14,8 +14,8 @@ import (
"github.com/syndtr/goleveldb/leveldb/opt"
"github.com/syndtr/goleveldb/leveldb/util"
coreserver "cosmossdk.io/core/server"
corestore "cosmossdk.io/core/store"
"cosmossdk.io/store/v2"
storeerrors "cosmossdk.io/store/v2/errors"
)
@ -28,7 +28,7 @@ type GoLevelDB struct {
db *leveldb.DB
}
func NewGoLevelDB(name, dir string, opts store.DBOptions) (*GoLevelDB, error) {
func NewGoLevelDB(name, dir string, opts coreserver.DynamicConfig) (*GoLevelDB, error) {
defaultOpts := &opt.Options{
Filter: filter.NewBloomFilter(10), // by default, goleveldb doesn't use a bloom filter.
}

View File

@ -10,8 +10,8 @@ import (
"github.com/cockroachdb/pebble"
"github.com/spf13/cast"
coreserver "cosmossdk.io/core/server"
corestore "cosmossdk.io/core/store"
"cosmossdk.io/store/v2"
storeerrors "cosmossdk.io/store/v2/errors"
)
@ -28,7 +28,7 @@ func NewPebbleDB(name, dataDir string) (*PebbleDB, error) {
return NewPebbleDBWithOpts(name, dataDir, nil)
}
func NewPebbleDBWithOpts(name, dataDir string, opts store.DBOptions) (*PebbleDB, error) {
func NewPebbleDBWithOpts(name, dataDir string, opts coreserver.DynamicConfig) (*PebbleDB, error) {
do := &pebble.Options{
MaxConcurrentCompactions: func() int { return 3 }, // default 1
}

View File

@ -4,8 +4,8 @@
package db
import (
coreserver "cosmossdk.io/core/server"
corestore "cosmossdk.io/core/store"
"cosmossdk.io/store/v2"
)
var _ corestore.KVStoreWithBatch = (*RocksDB)(nil)
@ -19,7 +19,7 @@ func NewRocksDB(name, dataDir string) (*RocksDB, error) {
panic("rocksdb must be built with -tags rocksdb")
}
func NewRocksDBWithOpts(dataDir string, opts store.DBOptions) (*RocksDB, error) {
func NewRocksDBWithOpts(dataDir string, opts coreserver.DynamicConfig) (*RocksDB, error) {
panic("rocksdb must be built with -tags rocksdb")
}

View File

@ -77,8 +77,3 @@ func (opts *PruningOption) ShouldPrune(version uint64) (bool, uint64) {
return false, 0
}
// DBOptions defines the interface of a database options.
type DBOptions interface {
Get(string) interface{}
}

View File

@ -54,12 +54,7 @@ func (roa *Reader) Has(key []byte) (bool, error) {
}
func (roa *Reader) Get(key []byte) ([]byte, error) {
result, err := roa.rootStore.GetStateStorage().Get(roa.actor, roa.version, key)
if err != nil {
return nil, err
}
return result, nil
return roa.rootStore.GetStateStorage().Get(roa.actor, roa.version, key)
}
func (roa *Reader) Iterator(start, end []byte) (corestore.Iterator, error) {