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

Varun Thacker commented on SOLR-7887:
-------------------------------------

{quote}You mean you can't find it in that file, or can't find where it's 
defined? If the former, see line #87 of configuring-logging.adoc. If the 
latter, see solr-ref-guide/src/_config.yml.template and 
solr-ref-guide/build.xml (or maybe just look at SOLR-12118).
{quote}
Thanks Cassandra for pointing it out. Looks like this commit will break the 
intention of how SOLR-12118 was supposed to work.

Quick analysis:

As of current master ivy-versions.properties has this
{code:java}
org.log4j.major.version = 2.11
org.log4j.minor.version = 0
org.apache.logging.log4j.version = 
${org.log4j.major.version}.${org.log4j.minor.version}{code}
_config.yml.template has this 
{code:java}
ivy-log4j-version: "${ivyversions.org.log4j.major.version}"{code}
And here is the excerpt from configuring-logging.adoc 
{code:java}
Solr uses http://logging.apache.org/log4j/{ivy-log4j-version}/[Log4J version 
{ivy-log4j-version}] for logging which is configured using 
`server/resources/log4j2.xml`.{code}
The website [https://logging.apache.org/log4j/2.x/] has 2.x and not 2.11 . 
Earlier it used to be [https://logging.apache.org/log4j/1.2/] so this would 
have worked before the upgrade of Log4J

 

cc [~hossman] [~erickerickson]

 

> Upgrade Solr to use log4j2 -- log4j 1 now officially end of life
> ----------------------------------------------------------------
>
>                 Key: SOLR-7887
>                 URL: https://issues.apache.org/jira/browse/SOLR-7887
>             Project: Solr
>          Issue Type: Task
>            Reporter: Shawn Heisey
>            Assignee: Erick Erickson
>            Priority: Major
>             Fix For: 7.4
>
>         Attachments: SOLR-7887-WIP.patch, SOLR-7887-eoe-review.patch, 
> SOLR-7887-eoe-review.patch, SOLR-7887-followup_1.patch, SOLR-7887.patch, 
> SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch, 
> SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch, 
> SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch, SOLR-7887.patch
>
>
> The logging services project has officially announced the EOL of log4j 1:
> https://blogs.apache.org/foundation/entry/apache_logging_services_project_announces
> In the official binary jetty deployment, we use use log4j 1.2 as our final 
> logging destination, so the admin UI has a log watcher that actually uses 
> log4j and java.util.logging classes.  That will need to be extended to add 
> log4j2.  I think that might be the largest pain point to this upgrade.
> There is some crossover between log4j2 and slf4j.  Figuring out exactly which 
> jars need to be in the lib/ext directory will take some research.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to