Actually, in looking at the code I see that there is already a semaphore there. It is just being improperly released by the rollover method.
Ralph > On Jan 24, 2017, at 6:31 PM, Apache <[email protected]> wrote: > > I thought of that but it doesn’t help. The renames are synchronous and they > happen with each rollover and conflict with the compression. We have to > prevent the next rollover from starting before the async task finishes. > > Ralph > >> On Jan 24, 2017, at 2:55 PM, Matt Sicker <[email protected] >> <mailto:[email protected]>> wrote: >> >> A custom ExecutorService extension that only allows one task of each kind to >> be executed at a time or some other classifier could be an approach possibly? >> >> On 24 January 2017 at 15:53, Apache <[email protected] >> <mailto:[email protected]>> wrote: >> I ended up having to put a delay in the loop that logs events and increase >> the number of bytes in each file. Obviously, this isn’t a perfect solution, >> but I’m really not sure what we can do if the async task takes longer than >> the time it takes to schedule it again. I suppose one option would be to put >> in a guard that would prevent a subsequent rollover from occurring until the >> async tasks complete. >> >> Ralph >> >> > On Jan 24, 2017, at 1:03 PM, Apache <[email protected] >> > <mailto:[email protected]>> wrote: >> > >> > I am looking at the failures that are occurring in the >> > RollingAppenderSizeTest in the Jenkins build. I am working at modifying >> > the code so that the asynchronous tasks will complete before shutdown is >> > allowed to complete. But I am running into an issue I am not sure how to >> > resolve. The test is configured to have the log file roll over after 500 >> > bytes have been written. It seems that some of the compression algorithms >> > take longer than it takes to write 500 bytes, so after the maximum number >> > of files are reached the system is rolling over on top of files that are >> > in the process of being archived, so we get rename/move and delete >> > problems. >> > >> > Any ideas? >> > >> > Ralph >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: [email protected] >> > <mailto:[email protected]> >> > For additional commands, e-mail: [email protected] >> > <mailto:[email protected]> >> > >> > >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> <mailto:[email protected]> >> For additional commands, e-mail: [email protected] >> <mailto:[email protected]> >> >> >> >> >> -- >> Matt Sicker <[email protected] <mailto:[email protected]>> >
