fzakaria / slf4j-timbre

SLF4J binding for Clojure's Timbre
Eclipse Public License 1.0
94 stars 24 forks source link

Utilize timbre ns-filtering #33

Closed khardenstine closed 5 years ago

khardenstine commented 5 years ago

slf4j performance guidelines suggest users to run expensive logging initialization code behind Logger.isXEnabled predicates (and they do)

Every time TimbreLoggerAdapter invokes timbre/may-log?, it has a chance to short circuit expensive computation by providing timbre's ns-filters with the logger's ns-str.

rufoa commented 5 years ago

Thanks Karl! This looks good. Will merge soon

rufoa commented 5 years ago

Pushed to clojars as v0.3.13