From 38c8faf140cb756cbec45cdd146bab56da2ab6a9 Mon Sep 17 00:00:00 2001 From: Brian Simpson Date: Tue, 22 Oct 2013 03:18:26 -0400 Subject: [PATCH] Don't memoize campaign_history. campaign_history is used to look up traffic for links in batch and the return can be too large for memoize. The query is fast and only used by admins so not memoizing it shouldn't be a problem. --- r2/r2/models/traffic.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/r2/r2/models/traffic.py b/r2/r2/models/traffic.py index 297f795bd..ebf27131f 100644 --- a/r2/r2/models/traffic.py +++ b/r2/r2/models/traffic.py @@ -523,7 +523,6 @@ class TargetedClickthroughsByCodename(Base): return total_by_codename(cls, codenames) @classmethod - @memoize_traffic(time=3600) def campaign_history(cls, codenames, start, stop): return campaign_history(cls, codenames, start, stop) @@ -605,7 +604,6 @@ class TargetedImpressionsByCodename(Base): return total_by_codename(cls, codenames) @classmethod - @memoize_traffic(time=3600) def campaign_history(cls, codenames, start, stop): return campaign_history(cls, codenames, start, stop)