vegaprotocol / data-node

A rich API server for Vega Protocol
https://vega.xyz
Other
3 stars 1 forks source link

fix: remove pk constraint for checkpoints #683

Closed pscott31 closed 2 years ago

pscott31 commented 2 years ago

Closes vegaprotocol/data-node#682 and vegaprotocol/vega#5700

@wwestgarth says on slack:

it is possible to get two checkpoints for the same block. We take a checkpoint at a set interval, but also we take them when a withdrawal happens. If a withdraw happens to happen on the same block we would naturally take a checkpoint we’ll get two checkpoint events for the same block

It might be that core can not do this, but while it does, don't force checkpoints to be unique on block height.