Skip to content

Commit 41a1a82

Browse files
committed
B2B-2931: [Graphql Resolver Cache] improve reliability and developer awareness on cache keyspace
- Remove references to 'custom' factor provisioning
1 parent a1a2b44 commit 41a1a82

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

app/code/Magento/GraphQlResolverCache/Model/Resolver/Result/CacheKey/Calculator/Provider.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
use Magento\GraphQlResolverCache\Model\Resolver\Result\CacheKey\Calculator;
1313

1414
/**
15-
* Provides custom cache key calculators for the resolvers chain.
15+
* Provides cache key calculators for the resolvers chain.
1616
*/
1717
class Provider implements ProviderInterface
1818
{
@@ -62,7 +62,7 @@ private function initForResolver(ResolverInterface $resolver): void
6262
"GraphQL Resolver Cache key factors are not determined for {$resolverClass} or its parents."
6363
);
6464
} else {
65-
$runtimePoolKey = $this->generateCustomProvidersKey($factorProviders);
65+
$runtimePoolKey = $this->generateKeyFromFactorProviders($factorProviders);
6666
if (!isset($this->keyCalculatorInstances[$runtimePoolKey])) {
6767
$this->keyCalculatorInstances[$runtimePoolKey] = $this->objectManager->create(
6868
Calculator::class,
@@ -74,17 +74,17 @@ private function initForResolver(ResolverInterface $resolver): void
7474
}
7575

7676
/**
77-
* Generate runtime pool key from the set of custom providers.
77+
* Generate runtime pool key from the set of factor providers.
7878
*
79-
* @param array $customProviders
79+
* @param array $factorProviders
8080
* @return string
8181
*/
82-
private function generateCustomProvidersKey(array $customProviders): string
82+
private function generateKeyFromFactorProviders(array $factorProviders): string
8383
{
84-
if (empty($customProviders)) {
84+
if (empty($factorProviders)) {
8585
return '';
8686
}
87-
$keyArray = array_keys($customProviders);
87+
$keyArray = array_keys($factorProviders);
8888
sort($keyArray);
8989
return implode('_', $keyArray);
9090
}
@@ -117,7 +117,7 @@ private function getResolverClassChain(ResolverInterface $resolver): array
117117
}
118118

119119
/**
120-
* Get custom cache key factor providers for the given resolver object.
120+
* Get a list of cache key factor providers for the given resolver object.
121121
*
122122
* @param ResolverInterface $resolver
123123
* @return array|null

app/code/Magento/GraphQlResolverCache/Model/Resolver/Result/CacheKey/Calculator/ProviderInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
use Magento\GraphQlResolverCache\Model\Resolver\Result\CacheKey\Calculator;
1212

1313
/**
14-
* Interface for custom cache key calculator provider.
14+
* Interface for cache key calculator provider.
1515
*/
1616
interface ProviderInterface
1717
{

0 commit comments

Comments
 (0)