Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrades mempool to become fork aware #86

Merged
merged 21 commits into from
Dec 15, 2022
Merged

Upgrades mempool to become fork aware #86

merged 21 commits into from
Dec 15, 2022

Conversation

sgerbino
Copy link
Member

@sgerbino sgerbino commented Dec 9, 2022

Resolves #79.
Resolves #81.
Resolves #82.
Resolves #83.
Resolves #85.

Brief description

Upgrades mempool to be fork aware such transactions that find themselves on orphaned forks do not get lost.

Checklist

  • I have built this pull request locally
  • I have ran the unit tests locally
  • I have manually tested this pull request
  • I have reviewed my pull request
  • I have added any relevant tests

Demonstration

ctest -j9
Test project /Users/sgerbino/Projects/koinos-mempool/build/tests
    Start 1: koinos_mempool_tests-mempool_tests/mempool_basic_test
    Start 2: koinos_mempool_tests-mempool_tests/pending_transaction_pagination
    Start 3: koinos_mempool_tests-mempool_tests/pending_transaction_pruning
    Start 4: koinos_mempool_tests-mempool_tests/pending_transaction_dynamic_max_resources
    Start 5: koinos_mempool_tests-mempool_tests/fork_test
1/5 Test #1: koinos_mempool_tests-mempool_tests/mempool_basic_test ..........................   Passed    0.06 sec
2/5 Test #4: koinos_mempool_tests-mempool_tests/pending_transaction_dynamic_max_resources ...   Passed    0.06 sec
3/5 Test #3: koinos_mempool_tests-mempool_tests/pending_transaction_pruning .................   Passed    0.06 sec
4/5 Test #5: koinos_mempool_tests-mempool_tests/fork_test ...................................   Passed    0.09 sec
5/5 Test #2: koinos_mempool_tests-mempool_tests/pending_transaction_pagination ..............   Passed    0.85 sec

100% tests passed, 0 tests failed out of 5

Total Test time (real) =   0.87 sec

@sgerbino sgerbino requested a review from mvandeberg December 13, 2022 09:37
@sgerbino sgerbino marked this pull request as ready for review December 13, 2022 20:50
@mvandeberg mvandeberg merged commit b0e3c2d into master Dec 15, 2022
@mvandeberg mvandeberg deleted the 79-fork-awareness branch December 15, 2022 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment