aboutsummaryrefslogtreecommitdiff
path: root/src/libstore/misc.cc
diff options
context:
space:
mode:
authorEelco Dolstra <eelco.dolstra@logicblox.com>2016-04-19 18:50:15 +0200
committerEelco Dolstra <eelco.dolstra@logicblox.com>2016-04-19 18:52:53 +0200
commite0204f8d462041387651af388074491fd0bf36d6 (patch)
treeecd20759ce49499722d140d653c5678051bcdfc2 /src/libstore/misc.cc
parent608b0265e104b4a97f51e5745b1a32078770f3cf (diff)
Move path info caching from BinaryCacheStore to Store
Caching path info is generally useful. For instance, it speeds up "nix path-info -rS /run/current-system" (i.e. showing the closure sizes of all paths in the closure of the current system) from 5.6s to 0.15s. This also eliminates some APIs like Store::queryDeriver() and Store::queryReferences().
Diffstat (limited to 'src/libstore/misc.cc')
-rw-r--r--src/libstore/misc.cc17
1 files changed, 9 insertions, 8 deletions
diff --git a/src/libstore/misc.cc b/src/libstore/misc.cc
index 12472f017..5c284d1b9 100644
--- a/src/libstore/misc.cc
+++ b/src/libstore/misc.cc
@@ -35,12 +35,13 @@ void Store::computeFSClosure(const Path & path,
if (includeDerivers && isDerivation(path)) {
PathSet outputs = queryDerivationOutputs(path);
for (auto & i : outputs)
- if (isValidPath(i) && queryDeriver(i) == path)
+ if (isValidPath(i) && queryPathInfo(i)->deriver == path)
edges.insert(i);
}
} else {
- queryReferences(path, edges);
+ auto info = queryPathInfo(path);
+ edges = info->references;
if (includeOutputs && isDerivation(path)) {
PathSet outputs = queryDerivationOutputs(path);
@@ -48,10 +49,8 @@ void Store::computeFSClosure(const Path & path,
if (isValidPath(i)) edges.insert(i);
}
- if (includeDerivers) {
- Path deriver = queryDeriver(path);
- if (isValidPath(deriver)) edges.insert(deriver);
- }
+ if (includeDerivers && isValidPath(info->deriver))
+ edges.insert(info->deriver);
}
for (auto & i : edges)
@@ -189,8 +188,10 @@ Paths Store::topoSortPaths(const PathSet & paths)
parents.insert(path);
PathSet references;
- if (isValidPath(path))
- queryReferences(path, references);
+ try {
+ references = queryPathInfo(path)->references;
+ } catch (InvalidPath &) {
+ }
for (auto & i : references)
/* Don't traverse into paths that don't exist. That can