Currently, the update process inserts a count of ads found into the watchdog log on every cron run, even if the number of ads don't change from the last cron run. This creates a lot of noise in the watchdog list. (Yes, it was my code that created this issue... :D )

I propose that we show the ad count only when it changes.

I've attached a proposed patch. Please review and comment.

CommentFileSizeAuthor
textlinkads-reduce-watchdog-noise.patch1.58 KBmcurry

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

This seems like a reasonably change. I didn't test it, but it seems good to me.

mcurry’s picture

mcurry commented

I've been using this patch on my site for a long time (at least, up till the time I gave TLA the boot). It really helps cut the log noise.

pluginstla_chris’s picture

pluginstla_chris commented
Status: Needs review » Closed (won't fix)

removing 4.x support