bobhansen
bobhansen
Like @lion10243, we added some extensions to add structured data in spdlog and a formatter to emit the logs. We've pushed the source up at https://github.com/bobhansen/structured_spdlog#using-structured-spdlog In this implementation, we've...
Thanks! Yes, the context is supported for async loggers. There are three classes of data that are emitted: fields passed into the log call (copied into a std::vector for async...
Changes consistenthash's get() function from O(log n) to O(1), with a memory overhead of 6*n (where n is the number of virtual nodes). On an AWS t3.2xlarge dedicated instance, this...
> This is a breaking API change for anybody using this package. Also, this parameter isn't documented. Maybe make a new NewXxx func instead taking the new parameter and make...
Oh, my. That's embarrassing. I got back to your comments in the hour after you made them, then forgot to push them up. They're pushed now.
Oh, my. That doesn't inspire confidence in someone's changes, does it? I had run `go test` at the root of the project, assuming that it would run for the transitive...
@bradfitz Requested changes made. Would love to see this change included so I can drop my forked repo.
> adding json support is overkill For our use case, we needed to have dynamic structured logging ship out logs to a system like Prometheus, and for better or worse,...
> 3: context lifetime object > > ```c++ > { > auto ctx = logger::context({{"user_id", get_user_id()}}); > logger::info("has context"); > } > logger::info("doesn't have context"); > ``` > > the...
Fixes #46