fabriziofortino commented on code in PR #2925:
URL: https://github.com/apache/jackrabbit-oak/pull/2925#discussion_r3341055123
##########
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:
a running repository has effectively one `ElasticConnection` (see
ElasticIndexProviderService#activate). I have anyway changed the logic to use a
shared global executor, in case there would be any need to handle multiple
connections.
--
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]
