Hmm, may be I'm not understanding the intent of the proposed throttling and the problem we are trying to solve well. My understanding was that a flood of certain (any) kind of log can result in significant disk i/o and impact the system throughput and stability and the proposed throttling is to be resilient against such issues. For such issues, I'm not sure if there's a lot of value in supporting advanced customizations. If the scope of the requirement is beyond resiliency and if there's a legitimate use case where throttling when exceeding a rate of 10 occurrences/sec vs 5/sec matters for each given type of log message, then I guess that's a different problem altogether. Nevertheless, sounds like a good topic to discuss during the summit? On Monday, October 19, 2020, 08:23:12 AM PDT, Alan Carroll <solidwallofc...@verizonmedia.com.invalid> wrote: I strongly suspect we might want it per logging message. At a minimum I don't think all messages should be throttled in this way, which is also an answer to Leif' question "how do I know which one to use"? Use the one that throttles if you want throttling, use the other if you don't want throttling. If you don't know that, you might want to think more carefully about what you're logging.
On Mon, Oct 19, 2020 at 9:09 AM Sudheer Vinukonda <sudheervinuko...@yahoo.com.invalid> wrote: > Hmm..do you mean system wide value? We can still have the wrap around > frequency configurable, but unless we want it customized per log line (I’m > not sure if this is really needed?), a system wide config may still be > possible to configure, no? > > > On Oct 19, 2020, at 7:04 AM, Alan Carroll > <solidwallofc...@verizonmedia.com.invalid> wrote: > > > > My concern at this point is the lack of configurability of the throttle > > time. The underlying implementation supports it but by wrapping it > directly > > in the logging call, only the default value can be used. > >