accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Resolved] (ACCUMULO-4650) Use SLF4J parameterized logging instead of String concatenation
Date Sat, 10 Jun 2017 16:08:19 GMT


Christopher Tubbs resolved ACCUMULO-4650.
    Resolution: Duplicate

Closing as duplicate. Recommend this work pick up from the previous patch or pull request
( That work is quite stale, and may need to be
redone, if it's too far out-of-date or incomplete.

> Use SLF4J parameterized logging instead of String concatenation
> ---------------------------------------------------------------
>                 Key: ACCUMULO-4650
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Michael Miller
>            Priority: Minor
> The Simple Logging Facade for Java (SLF4J) allows performance improvement of logging
with use of parameterized "{}" log messages.  
> The following two lines will yield the exact same output. However, the second form will
outperform the first form by a factor of at least 30, in case of a disabled logging statement.
> {code}
> logger.debug("The new entry is "+entry+".");
> logger.debug("The new entry is {}.", entry);
> {code}
> See here for reference:
> It wouldn't hurt to wrap all debug log statements server side with isDebugEnabled() as

This message was sent by Atlassian JIRA

View raw message