<div dir="ltr">Hi Gena,<div><br></div><div>The method described for a disk-cached AsyncAppender of course makes perfect sense. I think it would make a good alternative to the current AsyncAppender for some scenarios and would be a solid addition to the current Logback appenders.</div>
<div><br></div><div>I agree that AsyncAppender does not handle network failures, but in my opinion that is the responsibility of the target appender since it best knows how to handle different failure cases. For instance, some appenders may choose to retry immediately or may round-robin to an alternate server, etc. </div>
<div><br></div><div>I agree completely that AsyncAppender has many uses not limited to remote log targets. Generally speaking, the purpose of AsyncAppender is the same for local and remote targets - to reduce the impact of target latency on application performance, whether that latency is network or disk or something else doesn't really matter. </div>
<div><br></div><div>I feel like we are speaking past each other a little bit in that I am not proposing any of the specific shutdown hooks discussed so far at a theoretical level (without being fully thought through or developed, they are of course flawed). Rather, I am proposing that my solution be implemented as a framework for Logback, contributors to Logback, and/or end users of Logback to build shutdown hooks that fulfill specific needs. Some of the theoretical hooks discussed could be implemented, knowing that they are not general solutions, but rather niche solutions that may be acceptable for some applications. Then, leveraging Logback's community of contributors (including you and me), more general solutions can be developed and included as part of Logback. </div>
<div><br></div><div>Based on this back and forth discussion, I don't think there is any resistance to your proposed solution as it has the potential to be a reasonably generic shutdown hook that could be applicable to many common use cases. Furthermore, it could be implemented as one of the pluggable classes usable by the ShutdownHookAction. If your implementation is as generic and effective in practice as you have described, it could actually be the default ShutdownHook implementation. If for some reason a user found that your solution was non-optimal for their application, they could implement (or work with the Logback community to implement) an alternate ShutdownHook that does meet their needs.</div>
<div><br></div><div>I hope this helps explain my position adequately - I don't see our two solutions as competing alternatives, but rather as complimentary components of the shutdown cleanup functionality of Logback. My portion provides a framework for creating shutdown hooks that cleanup the Logback context (possibly including some simple implementations) and your portion includes a more complex, but fairly general implementation of that shutdown hook framework. This gives us the best of both worlds - the flexibility to plug different hooks in if needed as well as a solid default implementation that should cover most use cases. This would seem to fit with the methodology of design that Logback generally uses where multiple implementations are possible and generally speaking a few options are included in the library, but a default implementation that covers most scenarios is also included in the library and, unless specified otherwise, is the preferred option.</div>
<div><br></div><div>Once again, thank you for the good discussion!</div><div><br></div><div>Regards,</div><div><br></div><div>Mike Reinhold</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 19, 2014 at 5:25 PM, Gena Makhomed <span dir="ltr"><<a href="mailto:gmm@csdoc.com" target="_blank">gmm@csdoc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 19.03.2014 16:51, Michael Reinhold wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I agree that logging should be reliable and fast, however this is not<br>
always possible with some targets - particular cloud logging services<br>
such as Loggly, Logentries, Papertrail, Splunk Storm, Loggr, etc.<br>
</blockquote>
<br></div>
However it is possible to make logging even to remote services<br>
reliable and fast, from application code point of view. Making<br>
process of logging to remote services true asynchronous:<br>
<br>
- write all log events to ordinary log file at local file system<br>
<br>
- in background (daemon) thread scan log files, check at remote<br>
service which files should be synced and in which offset, after<br>
this first "handshake" - just monitor tail of log file, and all<br>
new events asynchronously read from log file and send to remote<br>
log collection service, even fashionable cloud logging services<br>
<br>
In this case - logging events to log file and sending events<br>
to remote logging service are asynchronous, and without event<br>
lost even after fast application restart, even with big queue<br>
of not yet reported to remote service log events in the log.<br>
<br>
Algorithm the same as in case of AsyncAppender from logback,<br>
but as temporary storage for queued events used persistent storage<br>
(disk subsystem) instead of java heap - this is only one difference.<br>
<br>
Existing AsyncAppender not help in case of temporary network failure,<br>
you will get event lost, - there is not guaranteed log events delivery.<br>
<br>
With true asynchronous conversation with remote logging service,<br>
(with queue located at persistent storage service, not in java heap)<br>
all events queued in log file will be send to remote logging service,<br>
without lost in events delivery process. (fast, asynchronous, reliable)<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The AsyncAppender was created to prevent target latency from impacting<br>
application performance by buffering the events and processing them<br>
asynchronously.<br>
</blockquote>
<br></div>
Yes, and AsyncAppender even help in case of using RollingFileAppender,<br>
allow application to process client request at maximum available speed,<br>
with all logging work with log file executed in separate daemon thread.<br>
<br>
If not use AsyncAppender before RollingFileAppender - all logging<br>
work with underlying OutputStream are done in application thread,<br>
and time wasted for logging instead of client request processing.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You describe a number of situations in which the discussed ShutdownHook<br>
implementations don't work. I don't want to argue a bunch of<br>
hypothetical scenarios on the mailing list, these were just descriptions<br>
of how it is possible to extend the ShutdownHooks in Logback. Please<br>
understand, I am not proposing that those are the (or the only)<br>
ShutdownHooks that get implemented - this was merely attempting to<br>
describe how different shutdown hooks could be developed that best meet<br>
different needs. Additionally, it was intended to show how the user<br>
could develop something that meets their specific application needs in<br>
the event that one of the Logback provided ShutdownHooks is not<br>
sufficient. Those examples may have been simplistic, but more complex<br>
options are possible. For instance, an InstrumentationShutdownHook that<br>
adds instrumentation into the Logback API to determine when it is safe<br>
to shutdown. Or a HeapWalkerShutdownHook that investigates the JVM heap<br>
to find Logger references. Or a ThreadHeuristicShutdownHook that<br>
analyzes the threads present in the JVM to determine when the Logback<br>
shutdown hook is the last relevant thread remaining. The point is that<br>
the ShutdownHookAction framework can be used to extend Logback's<br>
built-in shutdown cleanup functionality, either by the Logback team<br>
itself, by contributors to the project, or by the end application developer.<br>
</blockquote>
<br></div>
Such approach imply minimally invasive changes to Logback<br>
and forcing users to write own non-trivial ShutdownHooks.<br>
<br>
If implement my proposed solution - user not need write any code at all.<br>
User just need to answer to one simple question: application shutdown<br>
means JVM shutdown (for example in case of UNIX service/daemon) or<br>
application shutdown possible without JVM shutdown (OSGi, webapp, etc)<br>
<br>
if application shutdown == JVM shutdown user add to logback.xml<br>
one line, for enabling built-in logback shutdown hook,<br>
which after activation turn logback from "maximum speed"<br>
to "maximum safe" mode of work, and in this case<br>
logback will be ready for forthcoming JVM shutdown.<br>
event lost in this case will be minimal, because<br>
after logback shutdown hook complete work -<br>
all events from AsyncAppender queue will be<br>
written to its underlying appender, queue size set to 0,<br>
and AsyncAppender will be switched into transparent sync mode.<br>
also, all appenders switched to immediateFlush = true mode.<br>
in this case each log event from any shutdown hook will be<br>
guaranteed to be written into log file.<br>
without any additional work from user.<br>
<br>
if application shutdown possible without JVM shutdown,<br>
user just need to activate proper context listener<br>
for graceful shutdown of logback via LoggerContext.stop<br>
<br>
if none on this two methods of graceful shutdown activated -<br>
logback can write warning about misconfiguration to StatusManager.<br>
<br>
for user - logback will work very easy,<br>
fast and reliable without any headache.<br>
<br>
call LoggerContext.stop only required for case<br>
when application shutdown != JVM shutdown,<br>
<br>
and if application shutdown == JVM shutdown<br>
calling LoggerContext.stop is not the best solution,<br>
because it can lead to logging events lost.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
For your scenario in particular: you say that you have shutdown hooks<br>
installed by both your code and by third party libraries you use,<br>
correct?<br>
</blockquote>
<br></div>
Yes.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Are the Shutdown hooks inserted by your code and by the<br>
libraries deterministic in terms of name or quantity?<br>
</blockquote>
<br></div>
No.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
If so, you could<br>
whitelist those threads call LoggerContext.stop when they have exited<br>
(which you can confirm during your shutdown hook via either the Thread<br>
API or the ThreadMXBean API). If the shutdown hooks are not<br>
deterministic in quantity or name, you can use the protected<br>
java.lang.<u></u>ApplicationShutdownHooks<br></div>
<<a href="http://stackoverflow.com/questions/6865408/i-need-to-list-the-hooks-registered-with-java-lang-applicationshutdownhooks" target="_blank">http://stackoverflow.com/<u></u>questions/6865408/i-need-to-<u></u>list-the-hooks-registered-<u></u>with-java-lang-<u></u>applicationshutdownhooks</a>><div class="">
<br>
class during your application startup to determine the hooks that are<br>
installed and find the names of those hooks. Then your shutdown hook can<br>
check the status of those threads and call LoggerContext.stop once they<br>
have finished processing (excluding the Logbcack Shutdown hook of<br>
course). Actually, after thinking this through, it is possible that this<br>
type of a shutdown hook will work for my application as well...<br>
</div></blockquote>
<br>
Yes, this is working solution, but it is very uncomfortable,<br>
because I need permanently monitor list of used shutdown hooks.<br>
and permamently update list of whitelisted shutdown hooks,<br>
which must exit before my logback shutdown hook should call<br>
LoggerContext.stop<br>
<br>
Better approach - just after starting my own shutdown hook<br>
get list of all application shutdown hooks and wait all rest<br>
shutdown hooks termination before calling LoggerContext.stop<br>
<br>
in this case - deadlock can be reached if any other shutdown hook<br>
use same approach and wait for all rest shutdown hooks completion.<br>
<br>
for almost always working solution - we need whitelist parameter<br>
for this shutdown hook, allowing to exclude other shutdown hook<br>
with same algorithm from logback shutdown hook wait list.<br>
<br>
yes, this work, with the same reliability as my proposal.<br>
<br>
Such approach imply minimally invasive changes to Logback,<br>
but need manual deadlock detection and manual deadlock prevention.<br>
this is small probability, but still it is headache and annoyance.<br>
also, ApplicationShutdownHooks - this is implementation details,<br>
on any other JVM your shutdown hook may not work correctly at all.<br>
<br>
My approach - deadlock not possible, not need whitelist,<br>
no headache, no annoyances, no reflection, work on any JVM,<br>
but need slightly more code changes of logback.<br>
<br>
compare, which variant is easier to implement and to use by users?<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Don't misunderstand what I am saying - I think your proposal has its<br>
merits and has the potential to improve the reliability in these<br>
specific scenarios that we are discussing. In fact, I feel that if your<br>
solution does work as proposed it would work very well in conjunction<br>
with my changes to AsyncAppender and my addition of the<br>
ShutdownHookAction.<br>
</blockquote>
<br></div>
if implement my proposal - ShutdownHookAction will be only one<br>
for all cases, where application shutdown == JVM shutdown,<br>
in this case we just not need many different shutdown hooks,<br>
because this one (switching logback into "ready to shutdown" mode)<br>
will work very well for all possible use cases.<div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
That being said, your proposal is not without its<br>
challenges. For instance, even after you have called your proposed<br>
prepareToShutdown method you still need to avoid calling<br>
LoggerContext.stop until all other threads/hooks that log events are<br>
finished with their work, otherwise log events may still be dropped<br>
since the loggers are stopped. The same timing & race condition problems<br>
still exist, they have just been moved from one point in code to another.<br>
</blockquote>
<br></div>
If application shutdown == JVM shutdown<br>
LoggerContext.stop not need to be called at all.<br>
<br>
After JVM shutdown and process exit - operating system<br>
close all open log files without assistance.<br>
<br>
Timing & race condition problems exists after all shutdown hooks<br>
finished if any other thread still try to write something in log file.<br>
Such event can be lost, but it also definitely will be lost<br>
if LoggerContext.stop already was called by your shutdown hook.<br>
<br>
So - it is no any difference here about timing & race condition.<br>
<br>
This, last events lost generally unavoidable at all.<div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Best regards,<br>
 Gena<br>
______________________________<u></u>_________________<br>
Logback-user mailing list<br>
<a href="mailto:Logback-user@qos.ch" target="_blank">Logback-user@qos.ch</a><br>
<a href="http://mailman.qos.ch/mailman/listinfo/logback-user" target="_blank">http://mailman.qos.ch/mailman/<u></u>listinfo/logback-user</a><br>
</div></div></blockquote></div><br></div>