I did not try it out, yet, but the code certainly looks like it serves my purpose. Thanks! Joern On 31.10.2011, at 21:46, ceki wrote:
Hi Joern,
Just added timeOfLastReset method to StatusChecker. See http://github.com/ceki/logback/commit/f28496ad3e5c63 for details.
Let me know if it all fits together. -- Ceki http://twitter.com/#!/ceki
On 26/10/2011 12:09 PM, Joern Huxhorn wrote:
Yes, it would.
Are you planning to check for the "Will reset and reconfigure context"? That wouldn't be very efficient... on the other hand, this isn't code where efficiency is really important.
I'd much prefer to have such a method in your codebase since it would be your task to fix it if the above message would ever change in the future. I'm not very fond of checking for externally defined "magic" string constants in my own code. Also, if you'd ever add such a resetTime property in the future, you could simply change the implementation of the timeOfLastReset() method to evaluate that property instead of parsing the status messages.
So yes, I'd be very fine with that proposal.
On 26.10.2011, at 11:26, ceki wrote:
How about if there was a method which returned the time of the last reset? Something like:
StatusChecker checker = new StatusChercker(loggerContext); long time = checker.timeOfLastReset();
Would the above work for you?
logback-dev mailing list logback-dev@qos.ch http://mailman.qos.ch/mailman/listinfo/logback-dev