10000 [Cache] Use sub-second accuracy for internal expiry calculations · symfony/symfony@b734ea5 · GitHub
[go: up one dir, main page]
Skip to content

Commit b734ea5

Browse files
nicolas-grekasnicolas-grekas
committed
[Cache] Use sub-second accuracy for internal expiry calculations
1 parent a81d3ba commit b734ea5

File tree

8 files changed

+18
-19
lines changed

8 files changed

+18
-19
lines changed

‎src/Symfony/Component/Cache/Adapter/AbstractAdapter.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,15 +65,13 @@ function ($key, $value, $isHit) use ($defaultLifetime) {
6565
$this->mergeByLifetime = \Closure::bind(
6666
function ($deferred, $namespace, &$expiredIds) use ($getId) {
6767
$byLifetime = array();
68-
$now = time();
68+
$now = microtime(true);
6969
$expiredIds = array();
7070

7171
foreach ($deferred as $key => $item) {
7272
if (null === $item->expiry) {
7373
$ttl = 0 < $item->defaultLifetime ? $item->defaultLifetime : 0;
74-
} elseif ($item->expiry > $now) {
75-
$ttl = $item->expiry - $now;
76-
} else {
74+
} elseif (0 >= $ttl = (int) ($item->expiry - $now)) {
7775
$expiredIds[] = $getId($key);
7876
continue;
7977
}

‎src/Symfony/Component/Cache/Adapter/ArrayAdapter.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public function getItems(array $keys = array())
8787
CacheItem::validateKey($key);
8888
}
8989

90-
return $this->generateItems($keys, time(), $this->createCacheItem);
90+
return $this->generateItems($keys, microtime(true), $this->createCacheItem);
9191
}
9292

9393
/**
@@ -115,7 +115,7 @@ public function save(CacheItemInterface $item)
115115
$value = $item["\0*\0value"];
116116
$expiry = $item["\0*\0expiry"];
117117

118-
if (null !== $expiry && $expiry <= time()) {
118+
if (null !== $expiry && $expiry <= microtime(true)) {
119119
$this->deleteItem($key);
120120

121121
return true;
@@ -131,7 +131,7 @@ public function save(CacheItemInterface $item)
131131
}
132132
}
133133
if (null === $expiry && 0 < $item["\0*\0defaultLifetime"]) {
134-
$expiry = time() + $item["\0*\0defaultLifetime"];
134+
$expiry = microtime(true) + $item["\0*\0defaultLifetime"];
135135
}
136136

137137
$this->values[$key] = $value;

‎src/Symfony/Component/Cache/Adapter/ProxyAdapter.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ function (CacheItemInterface $innerItem, array $item) {
7373
$item["\0*\0value"] = array("\x9D".pack('VN', (int) $metadata[CacheItem::METADATA_EXPIRY] - CacheItem::METADATA_EXPIRY_OFFSET, $metadata[CacheItem::METADATA_CTIME])."\x5F" => $item["\0*\0value"]);
7474
}
7575
$innerItem->set($item["\0*\0value"]);
76-
$innerItem->expiresAt(null !== $item["\0*\0expiry"] ? \DateTime::createFromFormat('U', $item["\0*\0expiry"]) : null);
76+
$innerItem->expiresAt(null !== $item["\0*\0expiry"] ? \DateTime::createFromFormat('U', (int) $item["\0*\0expiry"]) : null);
7777
},
7878
null,
7979
CacheItem::class
@@ -192,7 +192,7 @@ private function doSave(CacheItemInterface $item, $method)
192192
}
193193
$item = (array) $item;
194194
if (null === $item["\0*\0expiry"] && 0 < $item["\0*\0defaultLifetime"]) {
195-
$item["\0*\0expiry"] = time() + $item["\0*\0defaultLifetime"];
195+
$item["\0*\0expiry"] = microtime(true) + $item["\0*\0defaultLifetime"];
196196
}
197197
$innerItem = $item["\0*\0poolHash"] === $this->poolHash ? $item["\0*\0innerItem"] : $this->pool->getItem($this->namespace.$item["\0*\0key"]);
198198
($this->setInnerItem)($innerItem, $item);

‎src/Symfony/Component/Cache/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ CHANGELOG
44
4.2.0
55
-----
66

7+
* added sub-second expiry accuracy for backends that support it
78
* added `CacheInterface` and `TaggableCacheInterface`, providing stampede protection via probabilistic early expiration
89
* throw `LogicException` when `CacheItem::tag()` is called on an item coming from a non tag-aware pool
910
* deprecated `CacheItem::getPreviousTags()`, use `CacheItem::getMetadata()` instead

‎src/Symfony/Component/Cache/CacheItem.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,9 @@ public function set($value)
8989
public function expiresAt($expiration)
9090
{
9191
if (null === $expiration) {
92-
$this->expiry = $this->defaultLifetime > 0 ? time() + $this->defaultLifetime : null;
92+
$this->expiry = $this->defaultLifetime > 0 ? microtime(true) + $this->defaultLifetime : null;
9393
} elseif ($expiration instanceof \DateTimeInterface) {
94-
$this->expiry = (int) $expiration->format('U');
94+
$this->expiry = (float) $expiration->format('U.u');
9595
} else {
9696
throw new InvalidArgumentException(sprintf('Expiration date must implement DateTimeInterface or be null, "%s" given', is_object($expiration) ? get_class($expiration) : gettype($expiration)));
9797
}
@@ -105,11 +105,11 @@ public function expiresAt($expiration)
105105
public function expiresAfter($time)
106106
{
107107
if (null === $time) {
108-
$this->expiry = $this->defaultLifetime > 0 ? time() + $this->defaultLifetime : null;
108+
$this->expiry = $this->defaultLifetime > 0 ? microtime(true) + $this->defaultLifetime : null;
109109
} elseif ($time instanceof \DateInterval) {
110-
$this->expiry = (int) \DateTime::createFromFormat('U', time())->add($time)->format('U');
110+
$this->expiry = microtime(true) + \DateTime::createFromFormat('U', 0)->add($time)->format('U.u');
111111
} elseif (\is_int($time)) {
112-
$this->expiry = $time + time();
112+
$this->expiry = $time + microtime(true);
113113
} else {
114114
throw new InvalidArgumentException(sprintf('Expiration date must be an integer, a DateInterval or null, "%s" given', is_object($time) ? get_class($time) : gettype($time)));
115115
}

‎src/Symfony/Component/Cache/Simple/ArrayCache.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public function getMultiple($keys, $default = null)
6464
CacheItem::validateKey($key);
6565
}
6666

67-
return $this->generateItems($keys, time(), function ($k, $v, $hit) use ($default) { return $hit ? $v : $default; });
67+
return $this->generateItems($keys, microtime(true), function ($k, $v, $hit) use ($default) { return $hit ? $v : $default; });
6868
}
6969

7070
/**
@@ -121,7 +121,7 @@ public function setMultiple($values, $ttl = null)
121121
}
122122
}
123123
}
124-
$expiry = 0 < $ttl ? time() + $ttl : PHP_INT_MAX;
124+
$expiry = 0 < $ttl ? microtime(true) + $ttl : PHP_INT_MAX;
125125

126126
foreach ($valuesArray as $key => $value) {
127127
$this->values[$key] = $value;

‎src/Symfony/Component/Cache/Tests/Fixtures/ArrayCache.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@ protected function doContains($id)
2121

2222
$expiry = $this->data[$id][1];
2323

24-
return !$expiry || time() <= $expiry || !$this->doDelete($id);
24+
return !$expiry || microtime(true) <= $expiry || !$this->doDelete($id);
2525
}
2626

2727
protected function doSave($id, $data, $lifeTime = 0)
2828
{
29-
$this->data[$id] = array($data, $lifeTime ? time() + $lifeTime : false);
29+
$this->data[$id] = array($data, $lifeTime ? microtime(true) + $lifeTime : false);
3030

3131
return true;
3232
}

‎src/Symfony/Component/Cache/Traits/ArrayTrait.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public function hasItem($key)
4444
{
4545
CacheItem::validateKey($key);
4646

47-
return isset($this->expiries[$key]) && ($this->expiries[$key] >= time() || !$this->deleteItem($key));
47+
return isset($this->expiries[$key]) && ($this->expiries[$key] >= microtime(true) || !$this->deleteItem($key));
4848
}
4949

5050
/**

0 commit comments

Comments
 (0)
0