Substrate: Allow full and light client on-disk databases to co-exist

Created on 12 Aug 2020  路  5Comments  路  Source: paritytech/substrate

This is a nice small issue to give the light client some love. If you run the full node for a bit, then a light node with --light, you get the following error:

Error: Service(Client(Backend("Unexpected database type. Expected: light")))

It's easy enough to change the paths where the databases are put, it might be best to put the full db in /db/full/ and the light db in /db/light.

P5-sometimesoon Q0-trivial

Most helpful comment

It's easy enough to change the paths where the databases are put, it might be best to put the full db in /db/full/ and the light db in /db/light.

(Pinging to get a consensus) @seunlanlege @rakanalh @danforbes @gnunicorn I believe this would be the best way to things, anyone else have an opinion?

All 5 comments

It's also important to consider that moving the path of the full db means that you have to add code that migrates from the old path.

It's easy enough to change the paths where the databases are put, it might be best to put the full db in /db/full/ and the light db in /db/light.

(Pinging to get a consensus) @seunlanlege @rakanalh @danforbes @gnunicorn I believe this would be the best way to things, anyone else have an opinion?

I think this is worth doing as i imagine there would be use cases where one would want to switch between the two.

+1

Sounds good to me, however, duplicate of https://github.com/paritytech/substrate/issues/2204 but the description is better in this issue so feel free to close https://github.com/paritytech/substrate/issues/2204

Was this page helpful?
0 / 5 - 0 ratings