[ 
https://issues.apache.org/jira/browse/SOLR-15610?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris M. Hostetter updated SOLR-15610:
--------------------------------------
    Attachment: SOLR-15610.patch
        Status: Open  (was: Open)

Thanks for the eyeballs Dawid - since the weirdness with {{versions.lock}} is 
evidently an existing issue, and not specific to something i did wrong when 
adding this jar, I've opened SOLR-15614 to track it independently.

(Since {{log4j-layout-template-json}} is released in lock step with the other 
{{log4j}} artifacts, and depends on {{log4j-core}}, I don't see how it could 
cause any problems in practice)
----
I'm attaching an updated patch that modifies the (existing) {{test_log4j}} 
docker test to demonstrate that {{<JsonTemplateLayout ... />}} can be used at 
runtime.

> 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
>         Attachments: SOLR-15610.patch, SOLR-15610.patch
>
>
> 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: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to