<div dir="ltr">Hi Ceki,<div><br></div><div>No problem! Things are busy all around. :) </div><div><br></div><div>I will rebase and separate this into two separate pull requests to make it easier to review and merge. I should be able to get this back to you before Monday without an issue. </div>
<div><br></div><div>Good catch on the ContextBase.removeShutdownHook - that reference should be cleared as well. I'll make this change in the new pull request. </div><div><br></div><div>Cheers,</div><div><br></div><div>
Mike Reinhold</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 4, 2014 at 2:43 AM, Ceki Gulcu <span dir="ltr"><<a href="mailto:ceki@qos.ch" target="_blank">ceki@qos.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Just looked at the code of ShutdownHookAction. It looks good.<br>
<br>
As for ContextBase.<u></u>removeShutdownHook, after calling Runtime.getRuntime().<u></u>removeShutdownHook(hook) you would also want to clear the hook associated with CoreConstants.SHUTDOWN_HOOK_<u></u>THREAD.<br>
<br>
Cheers,<br>
--<br>
Ceki<div class="HOEnZb"><div class="h5"><br>
<br>
On 04.04.2014 08:38, Ceki Gulcu wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Michael,<br>
<br>
Thank you for your message and my apologies for not responding sooner on<br>
this issue. Separating AsyncAppender and ShutdownHookAction would be<br>
nice but not strictly necessary. However, it would be easier to perform<br>
the merge after a rebase and a new pull request. Thank you for offering.<br>
Much appreciated.<br>
<br>
Best regards,<br>
--<br>
Ceki<br>
<br>
<br>
On 02.04.2014 22:47, Michael Reinhold wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Ceki,<br>
<br>
Glad to hear the next release of Logback is out!<br>
<br>
I apologize that I was not able to complete the changes for the<br>
ShutdownHookAction and associated components prior to this release. I<br>
did intend to have it ready for you to merge prior to 1.1.2 - clearly I<br>
missed! I just wanted to reach out to you to find out how best to<br>
proceed with getting the changes to AsyncAppender and the ShutdownHook<br>
merged into logback.<br>
<br>
I'm assuming that you would prefer that I pull and rebase my work<br>
against the latest commit in the repo? I'm not sure if I should close<br>
the current pull request and initiate a new pull request? Would you<br>
prefer that I split out the AsyncAppender changes into its own pull<br>
request, separate from the ShutdownHookAction components? This probably<br>
makes sense since the AsyncAppender timeout period is complete and<br>
valuable to others even without a built-in Shutdown hook mechanism in<br>
Logback.<br>
<br>
Please let me know how you would like me to proceed - I want to make<br>
including these features as easy for the Logback team as possible!<br>
<br>
Regards,<br>
<br>
Mike Reinhold<br>
<br>
<br>
</blockquote>
<br>
<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>
</blockquote>
<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>