Re: ExecutorService.shutdown

2017-01-11 Thread Mikael Ståldal
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" wrote: > > > On Jan 10, 2017, at 2:09 PM, Mikael Ståldal > wrote: > > > > It seems like ExecutorServ

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread JIRA
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819147#comment-15819147 ] Mikael Ståldal commented on LOG4J2-1748: But I think the proper way to handle thi

[jira] [Commented] (LOG4J2-1780) LoggerContext does not shut down old executor services when creating new ones during reconfiguration

2017-01-11 Thread JIRA
[ https://issues.apache.org/jira/browse/LOG4J2-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819168#comment-15819168 ] Mikael Ståldal commented on LOG4J2-1780: Otherwise, an easy solution would be to

[jira] [Commented] (LOG4J2-1776) log4j-boot pom modules for dependency management

2017-01-11 Thread JIRA
[ https://issues.apache.org/jira/browse/LOG4J2-1776?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819217#comment-15819217 ] Mikael Ståldal commented on LOG4J2-1776: Why do we have logback here? > log4j-bo

Re: How about a log4j-starters sort of repository?

2017-01-11 Thread Mikael Ståldal
I think that we should still split up log4j-core though. On Jan 10, 2017 3:18 AM, "Matt Sicker" wrote: > Going to go with the name Log4j Boot. I submitted a git repo request and > the form says it should be created within the hour. > > On 8 January 2017 at 23:13, Apache wrote: > >> Go for it! >

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819271#comment-15819271 ] Ralph Goers commented on LOG4J2-1748: - I don't know why those two ThreadPoolExecutors

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819273#comment-15819273 ] Ralph Goers commented on LOG4J2-1748: - I don't think that is a great recommendation.

[jira] [Comment Edited] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819273#comment-15819273 ] Ralph Goers edited comment on LOG4J2-1748 at 1/11/17 9:44 PM: -

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819275#comment-15819275 ] Ralph Goers commented on LOG4J2-1748: - Again, lets just get rid of these ExecutorServ

log4j-samples

2017-01-11 Thread Apache
I think we should make a separate repo for log4j-samples. The build of this is not part of our release process so it really makes no sense, other than making sure it builds, to have it be part of the main build. Thoughts? Ralph -

[jira] [Commented] (LOG4J2-1780) LoggerContext does not shut down old executor services when creating new ones during reconfiguration

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819309#comment-15819309 ] Ralph Goers commented on LOG4J2-1780: - The LoggerContext should not have any Executor

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Gary Gregory (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819329#comment-15819329 ] Gary Gregory commented on LOG4J2-1748: -- Using executor services lets us centralize t

[jira] [Commented] (LOG4J2-1780) LoggerContext does not shut down old executor services when creating new ones during reconfiguration

2017-01-11 Thread Gary Gregory (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819340#comment-15819340 ] Gary Gregory commented on LOG4J2-1780: -- Maybe the {{ConfigurationScheduler}} is misn

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819341#comment-15819341 ] Ralph Goers commented on LOG4J2-1748: - We have an ExecutorService. It is called Confi

[jira] [Commented] (LOG4J2-1780) LoggerContext does not shut down old executor services when creating new ones during reconfiguration

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819347#comment-15819347 ] Ralph Goers commented on LOG4J2-1780: - I don't see the problem. It is owned by the Co

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819604#comment-15819604 ] Remko Popma commented on LOG4J2-1748: - Probably best to have only one ExecutorService

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819620#comment-15819620 ] Remko Popma commented on LOG4J2-1748: - It would be nice if the shutdown hook would be

[jira] [Comment Edited] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819620#comment-15819620 ] Remko Popma edited comment on LOG4J2-1748 at 1/12/17 12:49 AM:

[jira] [Comment Edited] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15819620#comment-15819620 ] Remko Popma edited comment on LOG4J2-1748 at 1/12/17 12:52 AM:

[jira] [Commented] (LOG4J2-1748) RollingFile appender prevents a stand alone application to terminate for as long as 60 sec

2017-01-11 Thread Ralph Goers (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15820023#comment-15820023 ] Ralph Goers commented on LOG4J2-1748: - I have no problem using the ThreadFactory to c

[jira] [Commented] (LOG4J2-1381) Documentation for LoggerNameLevelRewritePolicy is wrong

2017-01-11 Thread Travis Spencer (JIRA)
[ https://issues.apache.org/jira/browse/LOG4J2-1381?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15820397#comment-15820397 ] Travis Spencer commented on LOG4J2-1381: While this area of the documentation is