diff options
| author |   Zac Medico <zmedico@gentoo.org> | 2008-06-04 01:23:05 +0000 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2008-06-04 01:23:05 +0000 |
| commit | d3d74ba390a663c5585090875fcb26d30ad3a824 (patch) | |
| tree | 8f141f47b04081a242254b6e7bd4fce7648128f3 /pym | |
| parent | Fit typo. (trunk r10566) (diff) | |
| download | portage-d3d74ba390a663c5585090875fcb26d30ad3a824.tar.gz portage-d3d74ba390a663c5585090875fcb26d30ad3a824.tar.bz2 portage-d3d74ba390a663c5585090875fcb26d30ad3a824.zip | |
Fix vardbapi.flush_cache() so that it only updates the cache when the
number of uncached packages reaches a certain threshold (currently 5).
The cache file can be several megabytes in size, so updating it for
every vdb change is wasteful. (trunk r10548)
svn path=/main/branches/2.1.2/; revision=10568
Diffstat (limited to 'pym')
| -rw-r--r-- | pym/portage.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/pym/portage.py b/pym/portage.py index 8ae940641..c61ed00cd 100644 --- a/pym/portage.py +++ b/pym/portage.py @@ -6727,6 +6727,10 @@ class vardbapi(dbapi): _excluded_dirs = re.compile(r'^(\..*|-MERGING-.*|' + \ "|".join(_excluded_dirs) + r')$') + # Number of uncached packages to trigger cache update, since + # it's wasteful to update it for every vdb change. + _aux_cache_threshold = 5 + _aux_multi_line_re = re.compile(r'^(CONTENTS|NEEDED\..*)$') """ @@ -7023,7 +7027,7 @@ class vardbapi(dbapi): users have read access and benefit from faster metadata lookups (as long as at least part of the cache is still valid).""" if self._aux_cache is not None and \ - self._aux_cache["modified"] and \ + self._aux_cache["modified"] >= self._aux_cache_threshold and \ secpass >= 2: valid_nodes = set(self.cpv_all()) for cpv in self._aux_cache["packages"].keys(): @@ -7038,7 +7042,7 @@ class vardbapi(dbapi): self._aux_cache_filename, gid=portage_gid, mode=0644) except (IOError, OSError), e: pass - self._aux_cache["modified"] = False + self._aux_cache["modified"] = 0 def aux_get(self, mycpv, wants): """This automatically caches selected keys that are frequently needed @@ -7080,7 +7084,7 @@ class vardbapi(dbapi): not self._aux_cache.get("packages"): self._aux_cache = {"version": self._aux_cache_version} self._aux_cache["packages"] = {} - self._aux_cache["modified"] = False + self._aux_cache["modified"] = 0 mydir = os.path.join(self.root, VDB_PATH, mycpv) mydir_stat = None try: @@ -7125,7 +7129,7 @@ class vardbapi(dbapi): for aux_key in cache_these: cache_data[aux_key] = mydata[aux_key] self._aux_cache["packages"][mycpv] = (mydir_mtime, cache_data) - self._aux_cache["modified"] = True + self._aux_cache["modified"] += 1 return [mydata[x] for x in wants] def _aux_get(self, mycpv, wants): |