mkapalka commented on code in PR #2925:
URL: https://github.com/apache/jackrabbit-oak/pull/2925#discussion_r3340655312


##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/ElasticConnection.java:
##########
@@ -70,6 +95,10 @@ public class ElasticConnection implements Closeable {
 
     private volatile Clients clients;
 
+    // dedicated executor to process async responses off the Elastic client 
I/O dispatcher thread (OAK-12174) without
+    // competing with callers that drive the query from the common pool 
(OAK-12234). Lazily created on first use.
+    private volatile ExecutorService responseExecutor;

Review Comment:
   We create `ElasticConnection` instances from a few locations: do we want to 
have a separate thread pool for each connection separately?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to