electrs icon indicating copy to clipboard operation
electrs copied to clipboard

Order History events in the same confirmation height.

Open junderw opened this issue 1 year ago • 3 comments

REQUIRES RE-INDEX

REQUIRES A LOT OF TESTING

This should fix the ordering of history entries and transactions within the same block/tx.

I think the addition of this data to the rows might cause unforseen consequences... so it needs to be tested thoroughly.

Requires a Re-Index.

junderw avatar May 07 '24 14:05 junderw

(Included #92 because mempool branch is broken without it)

junderw avatar May 07 '24 14:05 junderw

The behavior of the RocksDB is relied upon in weird ways, so the biggest worry I have is a re-surfacing of the weird balance-is-off bugs...

junderw avatar May 07 '24 14:05 junderw

(re)tested ACK @ [3dab4c7e]

I completed a full mainnet reindex with the new u16 tx position, and everything looks good so far, including balances on a bunch of heavily-trafficked addresses.

mononaut avatar May 13 '24 16:05 mononaut