WFLY-3888 enhance timer logging, show the full timer information for ERROR and WARN

cache the part of the string which is final and only replace the dynamic values.

https://issues.jboss.org/browse/WFLY-3888
No, just build the string using stringbuilder rather that using format()
On 02/10/2014 4:26 pm, "Wolf-Dieter Fink" <notifications@github.com> wrote:

> In ejb3/src/main/java/org/jboss/as/ejb3/timerservice/TimerImpl.java:
>
> > @@ -43,7 +43,14 @@
> >   * @version $Revision: $
> >   */
> >  public class TimerImpl implements Timer {
> > -
> > +    /**
> > +     * The output format for {@link #toString()}
> > +     */
> > +    private static final String FORMAT = "[id=%s timedObjectId=%s auto-timer?:%b persistent?:%b timerService=%s initialExpiration=%s intervalDuration(in milli sec)=%d nextExpiration=%%s timerState=%%s info=%s]";
>
> @stuartwdouglas <https://github.com/stuartwdouglas> that mean we should
> use the former toString() approach and just remove the TODO, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wildfly/wildfly/pull/6746/files#r18323933>.
>