[
https://issues.apache.org/jira/browse/SOLR-15610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17407666#comment-17407666
]
Chris M. Hostetter commented on SOLR-15610:
-------------------------------------------
The crux of this change would simply be...
{code:java}
diff --git a/solr/server/build.gradle b/solr/server/build.gradle
index 371c5ce456f..d6df58da7b0 100644
--- a/solr/server/build.gradle
+++ b/solr/server/build.gradle
@@ -72,6 +72,7 @@ dependencies {
libExt 'org.apache.logging.log4j:log4j-1.2-api'
libExt 'org.apache.logging.log4j:log4j-api'
libExt 'org.apache.logging.log4j:log4j-core'
+ libExt 'org.apache.logging.log4j:log4j-layout-template-json'
libExt 'org.apache.logging.log4j:log4j-slf4j-impl'
libExt 'org.apache.logging.log4j:log4j-web'
{code}
...but with the obvious additions of {{versions.lock}} and {{solr/licenses/}}
files for the new jar.
Any objections?
> Support json formatted logging via conf: add log4j-layout-template-json to
> 'server'
> -----------------------------------------------------------------------------------
>
> Key: SOLR-15610
> URL: https://issues.apache.org/jira/browse/SOLR-15610
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: Chris M. Hostetter
> Assignee: Chris M. Hostetter
> Priority: Major
>
> JSON Formatted logging is becoming wide spread enough that it seems like we
> should support it as an option in Solr.
> Assuming we upgrade to log4j >= 2.14.0 (SOLR-15609) then this becomes
> _possible_ for people to do by:
> # downloading the {{log4j-layout-template-json.jar}} that corresponds with
> the lof4j version used by their solr version
> # include the {{log4j-layout-template-json.jar}} in the (jetty) classpath
> via {{SOLR_OPTS}} (can't be loaded as a solr plugin because it needs to be
> used on startup)
> # specify their own {{LOG4J_PROPS}} file that configures log4j to use a
> {{JsonTemplateLayout}}
> ..but steps #1 and #2 are a bit hairy. Particularly step #1 as it means
> every time users upgrade solr they have to pay careful attention to if/when
> the log4j deps change.
> ----
> I would like to propose that we make this easier, by adding
> {{log4j-layout-template-json}} as a dependency for the "server" module, so
> that it is available out of the box when running solr – that way JSON logging
> can be enabled easily with a simple {{LOG4J_PROPS}} change.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
