atris commented on a change in pull request #916: LUCENE-8213: Asynchronous
Caching in LRUQueryCache
URL: https://github.com/apache/lucene-solr/pull/916#discussion_r330584691
##########
File path: lucene/core/src/java/org/apache/lucene/search/QueryCache.java
##########
@@ -33,4 +35,10 @@
*/
Weight doCache(Weight weight, QueryCachingPolicy policy);
+ /**
+ * Same as above, but allows passing in an Executor to perform caching
+ * asynchronously
+ */
+ Weight doCache(Weight weight, QueryCachingPolicy policy, Executor executor);
Review comment:
We cannot remove this constructor since it is defined in `QueryCache` --
made it directly delegate to the new constructor
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
