[
https://issues.apache.org/jira/browse/TAP5-1874?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jochen Kemnade updated TAP5-1874:
---------------------------------
Attachment: 0002-TAP5-1874-implement-the-variants-that-take-a-CharSeq.patch
0001-TAP5-1874-add-methods-that-take-a-CharSequence-for-t.patch
This is what I have in mind so far.
> OperationTracker description logging improvement
> ------------------------------------------------
>
> Key: TAP5-1874
> URL: https://issues.apache.org/jira/browse/TAP5-1874
> Project: Tapestry 5
> Issue Type: Improvement
> Components: tapestry-ioc
> Affects Versions: 5.3, 5.4
> Reporter: Denis Stepanov
> Assignee: Jochen Kemnade
> Priority: Minor
> Labels: desired_for_5.5
> Attachments:
> 0001-TAP5-1874-add-methods-that-take-a-CharSequence-for-t.patch,
> 0002-TAP5-1874-implement-the-variants-that-take-a-CharSeq.patch
>
>
> I like the concept of the OperationTracker, but it would be nice if the
> operation's description was created only when it's needed. Most of the
> tapestry code creates the description using the string concatenating or
> String.format all the time, but it is only used when an error occurs or the
> debug logging is enabled.
> I would suggest to create new interface Operation/OperationInvokable with
> getDescription method as the replacement of Runnable/Invokable interface:
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
