<p>Yes, the change to stop and asyncCompress would get things done. ;)</p>
<p>Thanks,<br>
-- TJ<br></p>
<p>Sent from mobile.</p>
<div class="gmail_quote">On Nov 6, 2011 2:27 PM, "ceki" <<a href="mailto:ceki@qos.ch" target="_blank">ceki@qos.ch</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
<br>
There is a rollover method in RollingFileAppender [1] and another one<br>
in the RollingPolicy interface. So the term rollover maybe ambiguous.<br>
<br>
I am under the impression that if the stop() method was modified to read:<br>
<br>
  public void stop() {<br>
      super.stop();<br>
      rollingPolicy.stop();<br>
      if(rollingPolicy != triggeringPolicy)<br>
        triggeringPolicy.stop();<br>
    }<br>
<br>
and if the access modifier for the asyncCompress method in<br>
TimeBasedRollingPolicy was changed to "protected", then the stop()<br>
method in your custom rolling policy could simply do the compression<br>
and the problem would be solved in just a few lines of code.<br>
<br>
Am I misreading an aspect of the problem?<br>
<br>
More below.<br>
--<br>
Ceki<br>
<br>
[1] <a href="http://goo.gl/ya7XK" target="_blank">http://goo.gl/ya7XK</a><br>
<br>
On 11/6/2011 7:20 PM, TJ Rothwell wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ceki,<br>
<br>
The compression is the goal. However a feature like rollover on close<br>
would make it easier to implement. The problem is that it adds<br>
complexity by modifying an interface that 3rd parties are using. With<br>
the base class, it reduces incompatibilities with those impls.<br>
<br>
Correct me if I'm wrong, but rollover does processing on the activefile<br>
inorder to prepare for the next activefile. The rollover process started<br>
from the appender doesn't create the next activefile. Its after rollever<br>
that the next activefile is created.<br>
</blockquote>
<br>
It depends on what you by rollover. As mentioned above there are two rollover methods. Were you aware of <a href="http://goo.gl/ya7XK" target="_blank">http://goo.gl/ya7XK</a> ?<br>
<br>
<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></div>