janhoy commented on a change in pull request #536:
URL: https://github.com/apache/solr/pull/536#discussion_r786634105



##########
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 found it. Default was 90 days, but I set the new default to 3 days to 
not flood someone's disks now that it becomes default.
   
   I also wrapped the default as a property, so you should be able to (not 
tested) set `-Dsolr.log.requestlog.retaindays=30` to configure the number of 
days you want to keep them. This needs to be documented in RefGuide.




-- 
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]

Reply via email to