From 5bb892b2063f5f795046ecf5f00a638b48bbbc9d Mon Sep 17 00:00:00 2001 From: Vincent Petry Date: Wed, 18 May 2016 13:49:21 +0200 Subject: Allow chunk GC mtime tolerance for unfinished part chunks Whenever part chunks are written, every fwrite in the write loop will reset the mtime to the current mtime. Only at the end will the touch() operation set the mtime to now + ttl, in the future. However the GC code is expecting that every chunk with mtime < now are old and must be deleted. This causes the GC to sometimes delete part chunks in which the write loop is slow. To fix this, a tolerance value is added in the GC code to allow for more time before a part chunk gets deleted. --- lib/private/cache/file.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/private/cache/file.php b/lib/private/cache/file.php index 3b500c4e45b..4624d3b2029 100644 --- a/lib/private/cache/file.php +++ b/lib/private/cache/file.php @@ -118,7 +118,9 @@ class File { public function gc() { $storage = $this->getStorage(); if($storage and $storage->is_dir('/')) { - $now = time(); + // extra hour safety, in case of stray part chunks that take longer to write, + // because touch() is only called after the chunk was finished + $now = time() - 3600; $dh=$storage->opendir('/'); if(!is_resource($dh)) { return null; -- cgit v1.2.3