lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Erick Erickson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SOLR-12549) Settle a location for the log4j2.xml file
Date Tue, 18 Sep 2018 14:49:00 GMT

    [ https://issues.apache.org/jira/browse/SOLR-12549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16619218#comment-16619218
] 

Erick Erickson commented on SOLR-12549:
---------------------------------------

Makes sense, but I pretty much guarantee I won't do it in time for getting it into 7.5. I'll
change it next time I have something to add in CHANGES.txt unless someone beats me to it.

> Settle a location for the log4j2.xml file
> -----------------------------------------
>
>                 Key: SOLR-12549
>                 URL: https://issues.apache.org/jira/browse/SOLR-12549
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: logging
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Major
>             Fix For: 7.5, master (8.0)
>
>         Attachments: SOLR-12008-7x-test-fail-fix.patch, SOLR-12008.patch, SOLR-12008.patch,
SOLR-12008.patch, SOLR-12008.patch, SOLR-12008.patch
>
>
> CLONED from SOLR-12008 since 12008 couldn't be closed.
> As part of SOLR-11934 I started looking at log4j.properties files. Waaay back in 2015,
the %C in "/solr/server/resources/log4j.properties" was changed to use %c, but the file in
"solr/example/resources/log4j.properties" was not changed. That got me to looking around and
there are a bunch of log4j.properties files:
> ./solr/core/src/test-files/log4j.properties
> ./solr/example/resources/log4j.properties
> ./solr/solrj/src/test-files/log4j.properties
> ./solr/server/resources/log4j.properties
> ./solr/server/scripts/cloud-scripts/log4j.properties
> ./solr/contrib/dataimporthandler/src/test-files/log4j.properties
> ./solr/contrib/clustering/src/test-files/log4j.properties
> ./solr/contrib/ltr/src/test-files/log4j.properties
> ./solr/test-framework/src/test-files/log4j.properties
> Why do we have so many? After the log4j2 ticket gets checked in (SOLR-7887) I propose
the logging configuration files get consolidated. The question is "how far"? 
> I at least want to get rid of the one in solr/example, users should use the one in server/resources.
Having to maintain these two separately is asking for trouble.
> [~markrmiller@gmail.com] Do you have any wisdom on the properties file in server/scripts/cloud-scripts?
> Anyone else who has a clue about why the other properties files were created, especially
the ones in contrib?
> And what about all the ones in various test-files directories? People didn't create them
for no reason, and I don't want to rediscover that it's a real pain to try to re-use the one
in server/resources for instance.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message