planetarium / libplanet

Blockchain in C#/.NET for on-chain, decentralized gaming
https://docs.libplanet.io/
GNU Lesser General Public License v2.1
505 stars 139 forks source link

Partially deprecate PoW `Block` handling #3790

Open greymistcube opened 1 month ago

greymistcube commented 1 month ago

There is no real code path that is actually taken within IActionEvaluator for past PoW Blocks.

That is, we are dragging along what is essentially a dead piece of code more for show (and our own emotional comfort) but not for real substance. However, the following two functionalities must be kept.

In order to do so,

[^1]: Note that write is not part of the spec, as this is no longer possible.

riemannulus commented 1 month ago

To be able to "read" PoW Blocks.

In post-sloth, IMO, we don't need able to read old PoW blocks. After sloth, we can easily cut out the block's height and It does not make lower chain security. So, instead of being able to read PoW blocks, how about a re-genesis on the NineChronicles and drop out old blocks?

greymistcube commented 1 month ago

Several things: