From 783294b4c61c180ac9bd9e5da8b0b4fc8dec449e Mon Sep 17 00:00:00 2001 From: Evan Vigil-McClanahan Date: Mon, 11 Apr 2022 16:29:58 -0700 Subject: [PATCH] use seek result, don't prefix key for seek --- src/ledger/v1/blockchain_ledger_v1.erl | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/ledger/v1/blockchain_ledger_v1.erl b/src/ledger/v1/blockchain_ledger_v1.erl index 1456b03865..1654ce61fe 100644 --- a/src/ledger/v1/blockchain_ledger_v1.erl +++ b/src/ledger/v1/blockchain_ledger_v1.erl @@ -2130,8 +2130,7 @@ promote_proposals(0, _Hash, _Height, _RandState, _Ledger, Iter, Acc) -> promote_proposals(K, BlockHash, BlockHeight, RandState, Ledger, Iter, Acc) -> {RandVal, NewRandState} = rand:uniform_s(RandState), RandHash = crypto:hash(sha256, <>), - rocksdb:iterator_move(Iter, {seek, <<3, RandHash/binary>>}), - NewAcc = case rocksdb:iterator_move(Iter, next) of + NewAcc = case rocksdb:iterator_move(Iter, {seek, RandHash}) of {ok, _Key, Binary} -> POC = blockchain_ledger_poc_v3:deserialize(Binary), ActivePOC0 = blockchain_ledger_poc_v3:status(active, POC), @@ -2142,7 +2141,6 @@ promote_proposals(K, BlockHash, BlockHeight, RandState, Ledger, Iter, Acc) -> {error, _} -> %% we probably fell off the end. Simply drop this as we may not have enough %% proposals to make the cut (or we can somehow retry some fixed number of times) - %% TODO check the iterator doesn't die here Acc end, promote_proposals(K - 1, BlockHash, BlockHeight, NewRandState, Ledger, Iter, NewAcc).