diff options
| author | Dan McGee <dan@archlinux.org> | 2012-11-27 08:48:01 -0600 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2012-11-27 08:48:01 -0600 |
| commit | eea25558c766d5f3a32879d16e579d051906cbf3 (patch) | |
| tree | c181d2ee27722ea1afb74990dd400202f9eeb5ec /main | |
| parent | a732d3cebcb8ff3170502b13d01ba90ac8efe26f (diff) | |
Don't cache package properties as aggressively
For package signatures, it turns out it is way cheaper to just parse the
signature again rather than going though all the decorator and
cache_function_key business. This speeds up the mismatched signatures
report significantly once this is removed.
For base_package, given that we only call it once from our package
details template, it makes little sense to cache the result.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'main')
| -rw-r--r-- | main/models.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/main/models.py b/main/models.py index cc81637c..603d7ccc 100644 --- a/main/models.py +++ b/main/models.py @@ -141,7 +141,6 @@ class Package(models.Model): return '%s://%s%s' % (proto, domain, self.get_absolute_url()) @property - @cache_function(15) def signature(self): try: data = b64decode(self.pgp_signature) @@ -154,7 +153,6 @@ class Package(models.Model): return packets[0] @property - @cache_function(15) def signer(self): sig = self.signature if sig and sig.key_id: @@ -318,7 +316,6 @@ class Package(models.Model): new_pkgs.append(package) return new_pkgs - @cache_function(125) def base_package(self): """ Locate the base package for this package. It may be this very package, |