8000 minor #37749 [Lock] downgrade log.info to log.debug (jderusse) · symfony/symfony@e1cfbd2 · GitHub
[go: up one dir, main page]

Skip to content

Commit e1cfbd2

Browse files
committed
minor #37749 [Lock] downgrade log.info to log.debug (jderusse)
This PR was merged into the 5.2-dev branch. Discussion ---------- [Lock] downgrade log.info to log.debug | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | Deprecations? | ni | Tickets | / | License | MIT | Doc PR | / This PR follows #37713 and reduce log verbosity in CombinedStore Commits ------- 080f114 Reduce log verbosity for CombinedStore
2 parents d66a0a7 + 080f114 commit e1cfbd2

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/Symfony/Component/Lock/Store/CombinedStore.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ public function save(Key $key)
6767
$store->save($key);
6868
++$successCount;
6969
} catch (\Exception $e) {
70-
$this->logger->warning('One store failed to save the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'exception' => $e]);
70+
$this->logger->debug('One store failed to save the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'exception' => $e]);
7171
++$failureCount;
7272
}
7373

@@ -82,7 +82,7 @@ public function save(Key $key)
8282
return;
8383
}
8484

85-
$this->logger->warning('Failed to store the "{resource}" lock. Quorum has not been met.', ['resource' => $key, 'success' => $successCount, 'failure' => $failureCount]);
85+
$this->logger->info('Failed to store the "{resource}" lock. Quorum has not been met.', ['resource' => $key, 'success' => $successCount, 'failure' => $failureCount]);
8686

8787
// clean up potential locks
8888
$this->delete($key);
@@ -103,15 +103,15 @@ public function putOffExpiration(Key $key, float $ttl)
103103
foreach ($this->stores as $store) {
104104
try {
105105
if (0.0 >= $adjustedTtl = $expireAt - microtime(true)) {
106-
$this->logger->warning('Stores took to long to put off the expiration of the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'ttl' => $ttl]);
106+
$this->logger->debug('Stores took to long to put off the expiration of the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'ttl' => $ttl]);
107107
$key->reduceLifetime(0);
108108
break;
109109
}
110110

111111
$store->putOffExpiration($key, $adjustedTtl);
112112
++$successCount;
113113
} catch (\Exception $e) {
114-
$this->logger->warning('One store failed to put off the expiration of the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'exception' => $e]);
114+
$this->logger->debug('One store failed to put off the expiration of the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'exception' => $e]);
115115
++$failureCount;
116116
}
117117

@@ -126,7 +126,7 @@ public function putOffExpiration(Key $key, float $ttl)
126126
return;
127127
}
128128

129-
$this->logger->warning('Failed to define the expiration for the "{resource}" lock. Quorum has not been met.', ['resource' => $key, 'success' => $successCount, 'failure' => $failureCount]);
129+
$this->logger->notice('Failed to define the expiration for the "{resource}" lock. Quorum has not been met.', ['resource' => $key, 'success' => $successCount, 'failure' => $failureCount]);
130130

131131
// clean up potential locks
132132
$this->delete($key);

0 commit comments

Comments
 (0)
0