Hi Tim,
Thanks for the new LoggingEvent.java file. As a side
note, please keep in mind that unified diffs (diff -u)
is the preferred way for suppying patches. This is
clearly not a big deal.
Coming back to the question of the timestamp, let me
say that the LoggingEvent class is designed with
safety and speed in mind, not flexiblity. This can be
considered as a design trade off (flaw?). It seems to
me that the fact that you could retrofit your
requirement to the existing design (with very little
modification) is due to your creativity and a pinch of
luck.
I am not sure other requirements could be added with
such ease. We should perhaps take the time to review
the design of the LoggingEvent class with more
flexibility in mind.
Paul Glezen is known to have opinions on the
subject.... Cheers, Ceki
--- Tim Dwelle <[EMAIL PROTECTED]> wrote:
> I have been using Log4J 1.1b6 to log messages that I
> pull
> off a JMS publish-subscribe topic. I'd like the
> timeStamp
> of the logging event to reflect the time the message
> was
> sent via JMS, rather than the time our subscriber
> gets
> around to logging the message.
>
> The current LoggingEvent derives the timeStamp when
> it is
> constructed, and that value cannot be changed (since
> the
> attribute is final). Attached is a modified version
> of
> LoggingEvent, with an alternate constructor allowing
> you
> to specify the timeStamp of the event.
>
> While I realize most people shouldn't need this...
> the
> change is extremely trivial and does make the system
> more
> flexible. And it would help me out a bunch!
>
> Thanks.
>
> - Tim Dwelle
> [EMAIL PROTECTED]
>
>
>
> ATTACHMENT part 2 application/octet-stream
name=LoggingEvent.java
>
---------------------------------------------------------------------
> To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> For additional commands, e-mail:
[EMAIL PROTECTED]
__________________________________________________
Do You Yahoo!?
Yahoo! Auctions - buy the things you want at great prices
http://auctions.yahoo.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
