<div dir="ltr">FYI -<div><br></div><div>I just closed the old pull request (177) that was based on 1.1.2. Pull request 186 is based on the 1.1.3 starting point commit and contains the changes to AsyncAppender as previously discussed in request 177. I'm working on rebasing and finishing the ShutdownHookAction changes and will submit that in a separate pull request as soon as I can.</div>
<div><br></div><div>Thanks!</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 9:20 AM, Michael Reinhold <span dir="ltr"><<a href="mailto:mike@coursescheduler.io" target="_blank">mike@coursescheduler.io</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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="HOEnZb"><div class="h5"><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><div><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>
</div></div></blockquote></div><br></div>