Qi Zhou

When the cluster shuts down (either gracefully or ungracefully), it is likely to enter a cluster inconsistent state, e.g., - a shard block is processed, and then the cluster is...

Not sure it is caused the rocksdb recovery, but the cluster hits inconsistent state after power cycle (1 out 3). We may need to add a feature to remove the...

Basically, add a run_cluster_from_snapshot.sh to do - If qkc-data folder exists, ask whether to replace existing qkc-data; and - Download the snapshot and unzip the snapshot to qkc-data; and -...

gas_price() estimate function may not return expected value for multi-native token

2

https://github.com/QuarkChain/pyquarkchain/blob/d470e13a26c64875652b3b1665eb58895baeb4ab/quarkchain/cluster/shard_state.py#L1777 The current estimation will estimate based on the historical gas price of the same token id, which may be empty or small if the token id is not frequently...

Cross-shard transaction could be fired after minor block observed the root block that creates a new shard

2

Currently, the logic requires the next root block observed, which is unnecessary.

ShardState.__validate_tx should not depend on shard state non-constant variable

2

https://github.com/QuarkChain/pyquarkchain/blob/7fb8b7cbeeb40dbef13ac29fc1c0eb40885a9ada/quarkchain/cluster/shard_state.py#L436-L448 As referenced by the above code, __validate_tx() uses root_tip to determine whether the neighbor exists or not, which is wrong for run_block() case, where the root block observed by...

integrity check of local db

4

Make sure if we change the code, the code can recognize local db, i.e., we don't break the existing consensus. A couple of checks: - Check every root block -...