Added guard for expensive debug logging

confluentinc / camus

Mirror of Linkedin's Camus

Apache License 2.0

0 stars 62 forks source link

Closed theduderog closed 9 years ago

theduderog commented 9 years ago

In my testing, adding a guard for this logging statement improved performance by about 40%.

ConfluentJenkins commented 9 years ago

Can one of the admins verify this patch?

ewencp commented 9 years ago

add to whitelist

ewencp commented 9 years ago

test this please

ewencp commented 9 years ago

Hmm, Jenkins seems to be misbehaving. Tested this the good old fashioned way and everything looks (unsurprisingly) good.

@theduderog Thanks for the fix, this is a good catch. Especially for relatively complex schemas, this will probably be a big performance win.