You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2018/11/01 21:02:01 UTC
[jira] [Commented] (GEODE-5686) Remove LocalizedStrings
[ https://issues.apache.org/jira/browse/GEODE-5686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16672191#comment-16672191 ]
ASF subversion and git services commented on GEODE-5686:
--------------------------------------------------------
Commit 75c40cf18b830b2213de0c5ef1ea731238d29d5a in geode's branch refs/heads/develop from [~bschuchardt]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=75c40cf ]
GEODE-5686: Remove LocalizedStrings
Fixing problems found by lgtm.com
There were a lot of logging calls that were improperly converted from
(MessageFormatString, Argument, Exception) to a form that treated the
Exception as part of the String.format() call.
There were a number of errors in format strings or number of arguments.
MessageFormat was more forgiving about this than String.format().
There were a few dangling MessageFormat specs that needed to be
converted to String.format() specs.
This closes #2757
> Remove LocalizedStrings
> -----------------------
>
> Key: GEODE-5686
> URL: https://issues.apache.org/jira/browse/GEODE-5686
> Project: Geode
> Issue Type: Improvement
> Components: logging
> Reporter: Bruce Schuchardt
> Assignee: Bruce Schuchardt
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> People have largely stopped using LocalizedStrings and we aren't using it to localize Geode anyway. Let's get rid of it!
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)