Closed dwasse closed 1 week ago
The code changes introduce a new parameter, maxPerChain
, which limits the number of transactions sorted by chain ID across various functions and tests. This parameter is utilized in the sortTxesByChainID
function to control the transaction count per chain, ensuring more granular sorting and processing. The changes span multiple files, updating function signatures and incorporating new logic to adhere to the maxPerChain
constraint.
Files | Change Summary |
---|---|
ethergo/submitter/export_test.go |
Updated SortTxes function to include maxPerChain parameter. |
ethergo/submitter/queue.go |
Updated processConfirmedQueue method to pass maxTxesPerChain constant to sortTxesByChainID . |
ethergo/submitter/util.go |
Enhanced sortTxesByChainID to accept maxPerChain and cap transactions accordingly. |
ethergo/submitter/util_test.go |
Modified SortTxes function in SubmitterSuite to accept a transaction cap. Added test cases for the new sorting behavior. |
sequenceDiagram
participant Caller
participant Queue
participant Util
Caller->>Queue: call processConfirmedQueue()
Queue->>Util: call sortTxesByChainID(txs, maxTxesPerChain)
Util-->>Queue: sortedTxes
Queue-->>Caller: processedTxes
In lines of code, we weave the change,
With new parameters to arrange. 🌟
Transactions sorted, capped with care,
Chain by chain, we plot and fare.
A dance of logic, swift and neat,
Our program's pulse, a steady beat. ✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 25.56528%. Comparing base (
07d6d4d
) to head (68029ac
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Latest commit: |
68029ac
|
Status: | ✅ Deploy successful! |
Preview URL: | https://a8800bbf.sanguine-fe.pages.dev |
Branch Preview URL: | https://feat-submitter-tx-cap.sanguine-fe.pages.dev |
Edget case: this will break if there's more than 100 atetmpts for a tx
Addresses #2750
Summary by CodeRabbit
New Features
Bug Fixes