Closed trajan0x closed 3 months ago
The recent changes integrate OpenTelemetry metrics recording into the ethergo/listener
package through a new otelRecorder
type and adjust concurrency handling in the Relayer
service by switching from TryAcquire
to Acquire
with a context parameter. Error messages and error handling have also been refined in various parts.
File(s) | Change Summary |
---|---|
ethergo/listener/listener.go |
Added otelRecorder to chainListener , initialized it in NewChainListener , and recorded metrics in doPoll . |
services/rfq/relayer/service/relayer.go |
Updated semaphore logic to use Acquire method with context parameter for better concurrency control. |
ethergo/listener/otel.go |
Introduced otelRecorder type with fields and methods for metrics tracking using OpenTelemetry. |
services/rfq/relayer/service/chainindexer.go |
Added defer statement in runChainIndexer to reset err to nil if it isn't nil . |
services/rfq/relayer/reldb/base/quote.go |
Updated error message in UpdateQuoteRequestStatus to use specific error type reldb.ErrNoQuoteForID . |
Changes in the code do chime,
Metrics trace the block’s rhyme. 🕰️
Semaphore now guards the way,
No moreTry
, context does sway.
Errors caught, specifics gleam,
In a world where data streams. 🚀
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 19.51220%
with 66 lines
in your changes missing coverage. Please review.
Project coverage is 25.38243%. Comparing base (
31b5f8f
) to head (60b0cf1
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[goreleaser]
Summary by CodeRabbit
New Features
Improvements
Bug Fixes
runChainIndexer
function to prevent unintended error propagation.