glyph at twistedmatrix.com
Sat Jun 15 20:38:31 MDT 2013
On Jun 15, 2013, at 9:17 AM, Christopher Armstrong <radix at twistedmatrix.com> wrote:
> I think I would just want one more feature:
>>>> Logger().emit(dudelog.LogLevel.warn, a="hi")
> Right now, this doesn't spit out anything into my log file, with
> default observers. I want it to emit some simple representation of the
> keys and values.
> On the other hand, the same is true of log.msg(a="hi"), but maybe this
> should be implemented (somehow) in Logger() codepath to avoid breaking
I keep going back and forth on this.
On the one hand, the main thing I want to do is to just say "if you want your message to appear in the log file, let's just implement a structured log file and you can read it from there. If you have no human-readable format string to explain your message, clearly you didn't intend for a human to read it."
On the other hand, we already implement log filtering, so filtering the message out of the log based on the absence of the format string is a little overloaded. And we should really be encouraging people to populate messages with useful structured data.
(I was going to say that without a format string I couldn't have my stand-in UUID feature, but then I realized that namespace + set of keys is probably good enough to generate that too, so never mind. Also it seems like I'm the only one who likes that feature so maybe it doesn't matter!)
How would you feel about a special token that you have to pass explicitly?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Twisted-Python