Differential D13068

[avalanche] Add transaction management to the processor core
ClosedPublic
Actions

Authored by Fabien on Jan 26 2023, 14:58.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

rABC1abd407a7ee5: [avalanche] Add transaction management to the processor core

Summary

This puts the transactions in parity with the blocks and proofs. Note the the sorting and filtering is dumb for now and needs to be expanded, which is not a problem since we don't add any transaction to reconcile yet.

Depends on D13066 and D13067.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_support_transaction_polls

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21882
Build 43399: Build Diff	build-debug · build-diff · lint-circular-dependencies · build-without-wallet · build-clang · build-clang-tidy
Build 43398: arc lint + arc unit

Event Timeline

Fabien created this revision.Jan 26 2023, 14:58

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 26 2023, 14:58

Fabien requested review of this revision.Jan 26 2023, 14:58

Harbormaster completed remote builds in B21882: Diff 37743.Jan 26 2023, 15:11

Cleanup before landing

This revision is now accepted and ready to land.Jan 26 2023, 17:05

sdulfari added inline comments.Jan 26 2023, 17:12

src/avalanche/test/processor_tests.cpp
381	Remove or rewrite

Closed by commit rABC1abd407a7ee5: [avalanche] Add transaction management to the processor core (authored by Fabien). · Explain WhyJan 26 2023, 19:41

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC1abd407a7ee5: [avalanche] Add transaction management to the processor core.

chill guy clicker is a great combination of creativity and relaxation, where you can create a wonderful world with just a few clicks, while enjoying a feeling of absolute peace.