picnoir/nix-gh
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

rename the options to mention it's a narinfo TTL as disk cache is used all over the place for other operations

Browse source
This commit is contained in:
AmineChikhaoui 2018-04-06 11:05:15 +01:00
parent 33b08899d5
commit 62d75ad3e1
No known key found for this signature in database
GPG key ID: C0C8C84C15BCCD1C
2 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/libstore/globals.hh
View file

@ -313,11 +313,11 @@ public:
Setting<Strings> trustedUsers{this, {"root"}, "trusted-users",
"Which users or groups are trusted to ask the daemon to do unsafe things."};
Setting<unsigned int> ttlNegativeDiskCache{this, 3600, "negative-disk-cache-ttl",
Setting<unsigned int> ttlNegativeNarInfoCache{this, 3600, "narinfo-cache-negative-ttl",
"The TTL in seconds for negative lookups in the disk cache i.e binary cache lookups that "
"return an invalid path result"};
Setting<unsigned int> ttlPositiveDiskCache{this, 30 * 24 * 3600, "positive-disk-cache-ttl",
Setting<unsigned int> ttlPositiveNarInfoCache{this, 30 * 24 * 3600, "narinfo-cache-positive-ttl",
"The TTL in seconds for positive lookups in the disk cache i.e binary cache lookups that "
"return a valid path result."};

8
src/libstore/nar-info-disk-cache.cc
View file

@ -113,8 +113,8 @@ public:
SQLiteStmt(state->db,
"delete from NARs where ((present = 0 and timestamp < ?) or (present = 1 and timestamp < ?))")
.use()
(now - settings.ttlNegativeDiskCache)
(now - settings.ttlPositiveDiskCache)
(now - settings.ttlNegativeNarInfoCache)
(now - settings.ttlPositiveNarInfoCache)
.exec();
debug("deleted %d entries from the NAR info disk cache", sqlite3_changes(state->db));
@ -183,8 +183,8 @@ public:
auto queryNAR(state->queryNAR.use()
(cache.id)
(hashPart)
(now - settings.ttlNegativeDiskCache)
(now - settings.ttlPositiveDiskCache));
(now - settings.ttlNegativeNarInfoCache)
(now - settings.ttlPositiveNarInfoCache));
if (!queryNAR.next())
return {oUnknown, 0};