Re: Improving the flexibility of RequestProgressTrackerLogFilter

2014-11-03 Thread Julian Sedding
Hi Bertrand Thank you for taking a look. As you observed correctly, the API changes were done to allow more flexibility when rendering the logs. The main objective was avoiding the date/time stamp (e.g. (2014-10-28 10:49:20)) in each line. Because it does not add any information, because the

Re: Improving the flexibility of RequestProgressTrackerLogFilter

2014-11-03 Thread Bertrand Delacretaz
Hi Julian, On Mon, Nov 3, 2014 at 10:15 AM, Julian Sedding jsedd...@gmail.com wrote: ...I have attached another patch to SLING-4114 that does not change the API. However, it drops the date/time from the message format... Cool, I like it, it's much better without API bundle changes. I have

Re: Improving the flexibility of RequestProgressTrackerLogFilter

2014-11-03 Thread Julian Sedding
Hi Bertrand Thanks a lot! I'm looking forward to using the new bundle. Regards Julian On Mon, Nov 3, 2014 at 4:26 PM, Bertrand Delacretaz bdelacre...@apache.org wrote: Hi Julian, On Mon, Nov 3, 2014 at 10:15 AM, Julian Sedding jsedd...@gmail.com wrote: ...I have attached another patch to

Re: Improving the flexibility of RequestProgressTrackerLogFilter

2014-10-31 Thread Bertrand Delacretaz
Hi Julian, On Wed, Oct 29, 2014 at 9:42 AM, Julian Sedding jsedd...@gmail.com wrote: ...I propose to reduce the size of the log file by doing the following: 1. changing the log format to include less redundant data 2. allow filtering by request duration 3. allow filtering by request

Improving the flexibility of RequestProgressTrackerLogFilter

2014-10-29 Thread Julian Sedding
Hi all The RequestProgressTrackerLogFilter allows logging the information collected by the RequestProgressTracker at debug level. Since this can amount to several undred lines logged per request, this may be prohibiitive on a busy production system. I propose to reduce the size of the log file