Page MenuHomePhabricator
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
sdulfari accepted this revision.Jan 26 2023, 17:05
sdulfari added a subscriber: sdulfari.

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 ↗(On Diff #37743)

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.

Revision Contents
Changeset List

PathSize
src/
avalanche/
19 lines
41 lines
test/
93 lines
4 lines
test/
2 lines

Diff 37749

View Options

src/avalanche/processor.h

Loading...
View Options

src/avalanche/processor.cpp

Loading...
View Options

src/avalanche/test/processor_tests.cpp

Loading...
View Options

src/init.cpp

Loading...
View Options

src/test/net_tests.cpp

Loading...