Reverting a block that consumed a source can delete the source, in SINDEX sub-index (indexForConditions)

This unit test failed "revert b#3-4 and re-commit block#3 should be ok"

Cause:

• In LevelDBSindex.removeBlock() (for reverting a block), we call trimConditions() when a consumption is undone (when the UPDATE record should be deleted) even if a source creation exists.

Solution:

• The correct approach would be to call trimConditions() only when no source creation exists anymore.

added D-medium label

assigned to @c-geek

dans la branche release/1.8 le test passait, mais simplement car getAvailableForPubkey() utilisait une findWhere() sur la table levelDB. Cependant, l'index indexForConditions était bien corrompu après un revert.

Je me suis rendu compte du problème pendant mes travaux d'optimisation de BMA, en modifiant getAvailableForPubkey() pour qu'il utilise indexForConditions. (cf MR)

mentioned in commit 3996d8f9

mentioned in merge request !1433 (closed)

mentioned in commit 992fdb0f

mentioned in commit bb828409

Vu. En fait, étant donné qu'une entrée UPDATE n'existe jamais toute seul dans le sindex, l'instruction if (!createRecord) conditions.push(updateRecord.conditions); du commit bb828409 pourrait même être complètement retirée.

Mais tu peux laisser comme cela.

Oui, je m'étais dis cela aussi. Merci !

unassigned @c-geek