accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ed Coleman (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3652) Remove string concatenation in log statements where slf4j is used.
Date Tue, 07 Apr 2015 12:27:12 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-3652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14483092#comment-14483092
] 

Ed Coleman commented on ACCUMULO-3652:
--------------------------------------

Bob, I went ahead an assigned this to you. As Christopher noted, this task has some overlap
with ACCUMULO-3681 (I have a current pull request that resolves that issue.) You should base
any work with ACCUMULO-3681 included to ease subsequent merge (I think the pull request is
being review and will be merged into the Accumulo master soon.)

With ACCUMULO-3681 I took the expedient path to resolve the issue - there is room to improve
some of the messages by supplying context rather than just repeating the exception message.
 

Let me know if you have any questions.

> Remove string concatenation in log statements where slf4j is used.
> ------------------------------------------------------------------
>
>                 Key: ACCUMULO-3652
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3652
>             Project: Accumulo
>          Issue Type: Sub-task
>          Components: build
>    Affects Versions: 1.7.0
>            Reporter: Ed Coleman
>            Assignee: Bob Thorman
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> As a separate task, continue with the conversion to remove log4j dependencies, modify
log statements where string concatenation is used and replace with {} parameter substitution.




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message