snarkOS icon indicating copy to clipboard operation
snarkOS copied to clipboard

Feature: get ciphertexts improvement

Open IAvecilla opened this issue 3 years ago • 3 comments

Motivation

Improve performance in get_ciphertexts method of LedgerState. This improvement was added in another PR to Testnet 3 #1759.

Test Plan

There are no new tests but the test_get_all_ciphertexts has been modified because the use of HashSet is no longer needed.

IAvecilla avatar May 06 '22 18:05 IAvecilla

@IAvecilla we'll need to resolve the merge conflict here

howardwu avatar May 25 '22 04:05 howardwu

@IAvecilla we'll need to resolve the merge conflict here

Done! 👍

IAvecilla avatar May 26 '22 12:05 IAvecilla

Admin: Please respond with "build this" in order to build this PR.

aleo-svc-exp avatar Aug 01 '22 23:08 aleo-svc-exp

These changes apply only for testnet2 because testnet3 has a big refactor on this logic, it will be better to do the necessary changes on the testnet3 if necessary

IAvecilla avatar Sep 22 '22 19:09 IAvecilla