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.

Sign up for GitHub

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

Jump to bottom

Sandbox (do not review): merge storage/txcache into Node #6684

Draft
wants to merge 225 commits into
base: feat/relayedv3
Choose a base branch
from
Draft

Sandbox (do not review): merge storage/txcache into Node #6684

wants to merge 225 commits into from

Conversation

andreibancioiu
Copy link
Collaborator

@andreibancioiu andreibancioiu commented Dec 18, 2024 •
edited
Loading

Steps involved:

git clone [email protected]:multiversx/mx-chain-go.git
git clone [email protected]:multiversx/mx-chain-storage-go.git

cd mx-chain-storage-go
git filter-repo --subdirectory-filter txcache --to-subdirectory-filter txcache

cd ../mx-chain-go

git checkout feat/relayedv3
git checkout -b merge-txcache
git remote add storage ../mx-chain-storage-go
git fetch storage
git merge --allow-unrelated-histories storage/feat/relayedv3
git remote remove storage
git push origin

How to review:

git clone -b merge-txcache [email protected]:multiversx/mx-chain-go.git
git clone -b feat/relayedv3 [email protected]:multiversx/mx-chain-storage-go.git
git diff --no-index mx-chain-go/txcache mx-chain-storage-go/txcache

References:

bogdan-rosianu and others added 30 commits August 8, 2022 14:15
@bogdan-rosianu
moved components a level behind
43d52cb
@bogdan-rosianu
fixes + refactoring
1c2431d
@bogdan-rosianu
moved errors and interfaces
045fbe8
@bogdan-rosianu
moved monitoring
efa9c2c
@AdoAdoAdo
remove pathmanager and directoryhandler
ca6d219 
move errors to common package
@AdoAdoAdo
Merge pull request #4 from ElrondNetwork/refactor-for-integration
fd83478 
Small refactor
@AdoAdoAdo
rebranding: rename module, adapt imports
ec39b3e
@AdoAdoAdo
Merge pull request #10 from multiversx/rebranding
38e9a4d 
rebranding: rename module, adapt imports
@AdoAdoAdo
rebranding: rename module, adapt imports
f780979
@AdoAdoAdo
Merge pull request #11 from multiversx/rebranding-fix-module
dd97337 
rebranding: rename module, adapt imports
@ssd04
Merge pull request #13 from multiversx/merge-main-into-rc-1.5.0
42ac8fa 
Merge main into rc 1.5.0
@AdoAdoAdo
Merge pull request #17 from multiversx/rc/v1.6.0
3120497 
RC/v1.6.0
@iulianpascalau
Merge pull request #36 from multiversx/merge-main-rc-v1.7.0-2024.01.29
bcb98eb 
Merge main rc v1.7.0 2024.01.29
@ssd04
Merge pull request #39 from multiversx/merge-rc-1.7.next1-to-common-p…
ec99f9c 
…ersisters

Merge rc 1.7.next1 to common persisters
@ssd04
Merge pull request #35 from multiversx/feat/refactor-persister-factory
deb30d5 
Feat/refactor persister factory
@iulianpascalau
Merge pull request #29 from multiversx/rc/v1.7.0
fea8551 
Rc/v1.7.0
@andreibancioiu
On NotifyAccountNonce, remove transactions with lower nonces.
525246c
@andreibancioiu
WrappedTransaction.computeFee() delegates to the Node's economicsData…
20a1b18 
… logic.
@andreibancioiu
Remove older feeComputationHelper.
ed82363
@andreibancioiu
New score function (avg ppu, scaled, logistic, re-scaled). Refactoring.
436ce87
@andreibancioiu
Integrate new score params in txListForSender.
a6ed2bd
@andreibancioiu
Implement noncesTracker. Able to tell if a nonces sequence is "spotle…
3adc37f 
…ss" (no gaps, no duplicates). Sketch some tests.
@andreibancioiu
Adjust tests.
471d581
@andreibancioiu
Adjust log levels.
f1e0364
@andreibancioiu
Adjust some tests.
d290551
@andreibancioiu
Fix score, fix tests.
fa036b2
@andreibancioiu
Simplification of sender score params.
e2db99a
@andreibancioiu
Fix score, add tests.
2259fea
@andreibancioiu
Optimizations.
ea5b28f
@andreibancioiu
Optimize "findInsertionPlace".
2511de6
andreibancioiu and others added 26 commits November 29, 2024 12:23
@andreibancioiu
Update description of isTransactionMoreValuableForNetwork.
679ff7e
@andreibancioiu
Fix typo.
cd11951
@andreibancioiu
Merge pull request #60 from multiversx/MX-16211-optimize-get-value
b40b0cc 
Mempool optimization: on transaction addition, call "GetTransferredValue" and retain its value
@andreibancioiu
Merge branch 'feat/mempool' into fix-txcache-readme
d2b8252
@andreibancioiu
Merge pull request #61 from multiversx/fix-txcache-readme
dde863e 
Mempool: fix readme about "isTransactionMoreValuableForNetwork"
@andreibancioiu
Merge branch 'main' into mempool-update-2024-11-29
b7e9d7a
@andreibancioiu
Fix linter issues.
20c155d
@andreibancioiu
Merge pull request #63 from multiversx/mempool-update-2024-11-29
27f0881 
Merge main into mempool, fix linter issues
@andreibancioiu
Add benchmarks for addition in worst case.
5189c96
@andreibancioiu
Fix linter issue.
dd5747b
@andreibancioiu
Merge pull request #64 from multiversx/MX-16202-sender-constraints
3f7ed6e 
Mempool: add benchmarks for addition in worst case
@andreibancioiu
Adjust benchmarks, lower maxNum when selecting transactions.
52d4374
@andreibancioiu
Adjust test config.
638e622
@andreibancioiu
Merge pull request #65 from multiversx/MX-16207-selection-max-num
a4ba77a 
Adjust benchmarks, lower "maxNum" when selecting transactions
@andreibancioiu
Merge pull request #62 from multiversx/feat/mempool
a3bc232 
Mempool improvements
@sstanculeanu
Merge pull request #66 from multiversx/rc/spica-patch-mempool
0bb822b 
Rc/spica patch mempool
@andreibancioiu
Sketch "accountsBalancesTracker".
8954e6a
@andreibancioiu
Decide fee payer (on wrapped transaction).
2f367bf
@andreibancioiu
Define a selection session wrapper to track accounts state, across al…
ccad4cd 
…l "heap items" (within a selection session).
@andreibancioiu
Add, fix unit tests. Fix eviction (undo change).
e90e9d3
@andreibancioiu
Add / improve tests.
83f8f36
@andreibancioiu
Additional tests, benchmarks.
3cf52ec
@andreibancioiu
Additional benchmarks.
82e71fa
@andreibancioiu
Fix after self review.
2c71521
@sstanculeanu
Merge pull request #67 from multiversx/MX-16294-selection-relayed
b6dc393 
Handle relayed V3 transactions in selection
@andreibancioiu
Merge remote-tracking branch 'storage/feat/relayedv3' into merge-txcache
1272a82
@andreibancioiu andreibancioiu self-assigned this Dec 18, 2024
@andreibancioiu andreibancioiu mentioned this pull request Dec 19, 2024
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@andreibancioiu andreibancioiu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@andreibancioiu @bogdan-rosianu @AdoAdoAdo @ssd04 @iulianpascalau @sstanculeanu