search

internetarchive / warcprox

WARC writing MITM HTTP/S proxy
371 stars 54 forks source link

tune MIN_BATCH_SEC, MAX_BATCH_SEC #173

Closed galgeek closed 2 years ago

wilsoniya commented 2 years ago

LGTM.

I'm interested to understand why it was necessary to tune this config. In my experience, the rationale behind tweaks like this is often valuable and worth adding as narrative in inline comments. Consider adding a few words explaining why we made this change and/or what it gets us. Thanks!

galgeek commented 2 years ago

Thanks, @wilsoniya — I've added an explanatory comment.