I would love it if ad_statistics became it's own module, so that we could turn off the excessive logging when it's not needed. None of the sites I use ad on actually need statistics tracking (since most of our ads come from ad services) but that ad_statisitcs table is always the largest table in my whole database. Seems such a waste!

Comments

jeremy’s picture

jeremy commented
Version: 6.x-3.x-dev » 6.x-2.x-dev

It should be simple enough to just add a configuration option to globally disable all statistics for the module, as all statistics are tracked in a central function. We'd need to confirm this would work with the ad_cache_* modules as well (likely simplest they'd be collected by the cache, but thrown away instead of being inserted into the database).

jeremy’s picture

jeremy commented

I took a quick look -- unfortunately what I said is not entirely true. The caching layers can insert into the database directly -- so to support this feature it would take changes in multiple places. Still, quite doable.

jenlampton’s picture

jenlampton
she / her
commented

Until we get this ironed out, I've created a tiny statistics killer: http://drupal.org/sandbox/jenlampton/1435124

But let me know when there's a patch to test :)

andrewtf’s picture

andrewtf commented

Flushing the ad statistics on cron could be really useful. However it would be preferable to be able to export or backup the statistics in a meaningful way (as a csv file or spreadsheet, for example) in case advertisers want to know how their ad's been performing. It actually doesn't happen too often, but of course the last time I truncated a too-huge ad statistics table via PHPmyadmin someone came of the woodwork and asked for a clickthrough number…

gregarios’s picture

gregarios commented

I'm all for improving the performance by having the option to totally disable click- and stat-tracking for all or some specific ads.

lrwebks’s picture

lrwebks commented
Issue summary: View changes
Status: Active » Closed (outdated)

Drupal 6 is EOL and no longer supported. Closing this as outdated for that reason. Thanks for your contribution!