Switch the wallet to the storage service - Githubissues

linera-io / linera-protocol

Main repository for the Linera protocol

Apache License 2.0

440 stars 126 forks source link

Switch the wallet to the storage service #2183

Open MathieuDutSik opened 3 months ago

MathieuDutSik commented 3 months ago

Motivation

The use of RocksDb for the wallet forces compiling with RocksDb which is bad for the CI.

Proposal

The following are done:

The RocksDb is no longer a default feature.
The default storage for the wallet switches from RocksDb to storage_service.
The path for the wallet is assigned to {path}_cliend_{id}.db which is maybe not a nice table name. But it works here.

Test Plan

The CI is the goal here.

Release Plan

Not relevant.

Links

reviewer checklist

afck commented 3 months ago

Maybe the default storage should still be RocksDB (if it's compiled in)? Other than in tests, I imagine we always want RocksDB?