move from slog to tracing

jondot commented 8 months ago

This PR moves from slog to tracing across the board.

Benefits:

Move to more "standardized" logging
Inversion of control - publisher-subscriber model, no longer need to pass logger around
All the power and flexibility of tracing and opentracing at hand (filtering, formatting, collection, shipping to OTEL, and more)
Integrate into production deployments better: one tracing filter/level to rule them all (i.e. filter across libraries such as sqlx)
Opens up the possibility of spans across jobs, components and advanced instrumentation
Removal of a good amount of code and an aging infrastructure (slog, log)

film42 commented 8 months ago

Wow! This is objectively way better. I want to merged this. I will probably push a few tweaks to demo code and README but this looks great. Thanks, @jondot !

film42 commented 8 months ago

Thanks for the thiserror change as well. I think it's a good change. Would prefer that to be a separate PR but directionally 👍 . I'm fine to keep this bundled in one PR. Hoping to get this released this weekend.

film42 commented 8 months ago

Release as v0.8.0. Thanks!

film42 / sidekiq-rs

move from slog to tracing #29