janhoy commented on a change in pull request #536:
URL: https://github.com/apache/solr/pull/536#discussion_r786715198
##########
File path: solr/server/etc/jetty-requestlog.xml
##########
@@ -30,7 +30,7 @@
<New class="org.eclipse.jetty.server.AsyncRequestLogWriter">
<Arg><Property name="solr.log.dir"
default="logs"/>/yyyy_mm_dd.request.log</Arg>
<Set name="filenameDateFormat">yyyy_MM_dd</Set>
- <Set name="retainDays">90</Set>
+ <Set name="retainDays"><Property
name="solr.log.requestlog.retaindays" default="3"/></Set>
Review comment:
I updated the refguide docs with the new defaults and system property. I
found it strange for the "Request logging" paragraph to live in "[Securing
Solr](https://solr.apache.org/guide/8_11/securing-solr.html#request-logging)"
page, so I also moved it to "[Configuring
logging](https://solr.apache.org/guide/8_11/configuring-logging.html)". Let me
know what you think.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
