You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@thrift.apache.org by GitBox <gi...@apache.org> on 2020/06/09 16:23:19 UTC

[GitHub] [thrift] belugabehr commented on pull request #2177: THRIFT-4926: Adding LOGGER.isDebugEnabled() conditions into TSaslTransport

belugabehr commented on pull request #2177:
URL: https://github.com/apache/thrift/pull/2177#issuecomment-640983004


   No changes provided address any kind of "leakage".  If the logging level is not set to DEBUG, then a DEBUG statement will not be emitted.  The `LOGGER.debug()` method checks internally for the appropriate condition.  The method `isDebugEnabled` is available only for performance reasons.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org