A) Sorry, ExecutorServices.java, line 51 and 76. B) You should discuss that with Gary, and join the discussion on LOG4J2-1748 which shows a weakness with this.
On Jan 11, 2017 6:25 AM, "Apache" <ralph.go...@dslextreme.com> wrote: > > > On Jan 10, 2017, at 2:09 PM, Mikael Ståldal <mikael.stal...@magine.com> > wrote: > > > > It seems like ExecutorService.shutdown invokes > executorService.shutdown() twice If timeout is 0. Is this correct? > > A) Do you mean ExecutorServices? I don’t see any code that calls > shutdown() twice. I do see code that calls shutdown() followed by > shutdownNow(). > B) Until you mentioned this I hadn’t even noticed that ExecutorServices > were added to the LoggerContext. I have no idea why that was done. The > whole point of the ConfigurationScheduler is to handle running asynchronous > tasks from one place. Those extra ExecutorServices really should be removed. > > Ralph > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: log4j-dev-unsubscr...@logging.apache.org > For additional commands, e-mail: log4j-dev-h...@logging.apache.org > >
