[ https://issues.apache.org/jira/browse/LOG4J2-1271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15149988#comment-15149988 ]
Remko Popma edited comment on LOG4J2-1271 at 2/17/16 7:54 AM: -------------------------------------------------------------- * postpone creating the formatted message until hand-off to other thread - makes sense * saving reference to the Throwable if any - makes sense * calling toString() on the parameters - *why do we need this?* I claim this becomes unnecessary once the formatted message has been created. was (Author: rem...@yahoo.com): * postpone creating the formatted message until hand-off to other thread - okay * saving reference to the Throwable if any - okay * calling toString() on the parameters - *not okay*: why do we need this? I claim this becomes unnecessary once the formatted message has been created. > ParameterizedMessage optimization > --------------------------------- > > Key: LOG4J2-1271 > URL: https://issues.apache.org/jira/browse/LOG4J2-1271 > Project: Log4j 2 > Issue Type: Improvement > Components: API > Affects Versions: 2.5 > Reporter: Remko Popma > Assignee: Remko Popma > > ParameterizedMessage creates unnecessary objects and does other unnecessary > work in its constructor. > On the other hand, it does not create the full formatted string it will > eventually need to return from {{getFormattedMessage()}}. > I propose the following changes: > * Don't call {{argumentsToStrings}} in the constructor. Converting the > parameter Object[] array to a String[] array should only be done when > necessary: when this ParameterizedMessage is serialized. > * Don't copy the parameter Object[] array to a new array. Instead clone the > array when the {{getParameters()}} method is called. > * In the constructor, build the fully formatted message. This combines the > work done by {{formatStringArgs(String, String[])}} and > {{argumentsToStrings(Object[])}} and avoids the issue mentioned in LOG4J2-763. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: log4j-dev-unsubscr...@logging.apache.org For additional commands, e-mail: log4j-dev-h...@logging.apache.org