tracing
tracing copied to clipboard
Add `record_bytes` to `Visit`
Motivation
We use a lot of data that's encoded as bytes that we would like to record to spans and events. While it's possible to use the debug representation, this is not so efficient.
Solution
Add record_bytes
method to the Visit
trait which defaults to use record_debug
in the same way as the existing methods.
Since this is a tracing-core
change, it would be nice to get @carllerche's approval on this.
Thanks - if the change is acceptable, I can add more documentation / examples.
@hawkw this one would be good to get in some day soon. Anything I can do to help expedite the review here?
:)
I don't have a problem w/ this.
This is still something that's going to be useful - anything I can do to move it along? Or any concerns?
Any news on when this might get merged?
After a rebase, this could probably be merged