Closed dwasse closed 2 weeks ago
In the chain_queue.go
file of the ethergo package, the significant changes include replacing the use of errgroup.Group
for concurrency with sync.WaitGroup
and modifying two methods (bumpTX
and updateOldTxStatuses
) to return errors. These changes aim to improve the handling of transaction bumping logic and error management in a more structured way without halting the entire queue due to individual errors.
Files | Change Summary |
---|---|
ethergo/submitter/chain_queue.go |
- Removed g *errgroup.Group field from the chainQueue struct- Added wg *sync.WaitGroup field to the chainQueue struct- Refactored bumpTX to return an error and restructured its logic- Refactored updateOldTxStatuses to return an error and handle updates in parallel |
Amidst the lines of struct and key, A queue transformed with sync, you see. No more halting, in graceful flight, Each transaction bumps with might. Errors handled, smooth as light, In CodeRabbit's world so bright. 🌟🐇
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?
Attention: Patch coverage is 18.27957%
with 76 lines
in your changes missing coverage. Please review.
Project coverage is 26.16388%. Comparing base (
3eccadb
) to head (788e685
).
Files | Patch % | Lines |
---|---|---|
ethergo/submitter/chain_queue.go | 18.27957% | 74 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Latest commit: |
788e685
|
Status: | ✅ Deploy successful! |
Preview URL: | https://9c89349c.sanguine-fe.pages.dev |
Branch Preview URL: | https://fix-bumptx-wg.sanguine-fe.pages.dev |
Addresses #2753
Summary by CodeRabbit