Optimise storing metadata in cache

doctrine · Dec 12, 2020 · c2e4044 · c2e4044
1 parent 118d9a6
commit c2e4044
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 3 deletions.
diff --git a/lib/Doctrine/Persistence/Mapping/AbstractClassMetadataFactory.php b/lib/Doctrine/Persistence/Mapping/AbstractClassMetadataFactory.php
@@ -12,6 +12,9 @@
 use Symfony\Component\Cache\Adapter\DoctrineAdapter;
 use Symfony\Component\Cache\DoctrineProvider;
 
+use function array_combine;
+use function array_keys;
+use function array_map;
 use function array_reverse;
 use function array_unshift;
 use function explode;
@@ -225,9 +228,18 @@ public function getMetadataFor($className)
 
                     $this->wakeupReflection($cached, $this->getReflectionService());
                 } else {
-                    foreach ($this->loadMetadata($realClassName) as $loadedClassName) {
-                        $item = $this->cache->getItem($this->getCacheKey($loadedClassName));
-                        $item->set($this->loadedMetadata[$loadedClassName]);
+                    $loadedMetadata = $this->loadMetadata($realClassName);
+                    $classNames     = array_combine(
+                        array_map([$this, 'getCacheKey'], $loadedMetadata),
+                        $loadedMetadata
+                    );
+
+                    foreach ($this->cache->getItems(array_keys($classNames)) as $item) {
+                        if (! isset($classNames[$item->getKey()])) {
+                            continue;
+                        }
+
+                        $item->set($this->loadedMetadata[$classNames[$item->getKey()]]);
                         $this->cache->saveDeferred($item);
                     }
 

diff --git a/tests/Doctrine/Tests/Persistence/Mapping/ClassMetadataFactoryTest.php b/tests/Doctrine/Tests/Persistence/Mapping/ClassMetadataFactoryTest.php
@@ -170,6 +170,10 @@ public function testWillIgnoreCacheEntriesThatAreNotMetadataInstances(): void
 
         $item = $this->createMock(CacheItemInterface::class);
 
+        $item
+            ->expects(self::any())
+            ->method('getKey')
+            ->willReturn($key);
         $item
             ->expects(self::any())
             ->method('get')
@@ -185,6 +189,11 @@ public function testWillIgnoreCacheEntriesThatAreNotMetadataInstances(): void
             ->method('getItem')
             ->with($key)
             ->willReturn($item);
+        $cacheDriver
+            ->expects(self::once())
+            ->method('getItems')
+            ->with([$key])
+            ->willReturn([$item]);
         $cacheDriver
             ->expects(self::once())
             ->method('saveDeferred')