arturobernalg commented on code in PR #466:
URL:
https://github.com/apache/httpcomponents-client/pull/466#discussion_r1267913105
##########
httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/BasicHttpCache.java:
##########
@@ -313,36 +324,64 @@ public CacheHit storeReusing(
return store(request, originResponse, requestSent, responseReceived,
rootKey, hit.entry);
}
- @Override
- public void flushCacheEntriesFor(final HttpHost host, final HttpRequest
request) {
- final String rootKey = cacheKeyGenerator.generateKey(host, request);
+ private void evictAll(final HttpCacheEntry root, final String rootKey) {
if (LOG.isDebugEnabled()) {
- LOG.debug("Flush cache entries: {}", rootKey);
+ LOG.debug("Evicting root cache entry {}", rootKey);
}
- try {
- storage.removeEntry(rootKey);
- } catch (final ResourceIOException ex) {
- if (LOG.isWarnEnabled()) {
- LOG.warn("I/O error removing cache entry with key {}",
rootKey);
+ removeInternal(rootKey);
+ if (root.isVariantRoot()) {
+ for (final String variantKey : root.getVariantMap().values()) {
+ LOG.debug("Evicting variant cache entry {}", variantKey);
Review Comment:
My suggestion was a general recommendation and aimed more at reducing log
noise when evicting entries.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
For additional commands, e-mail: dev-h...@hc.apache.org
