nit0906 commented on code in PR #490:
URL: https://github.com/apache/jackrabbit-oak/pull/490#discussion_r871560205
##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/index/ElasticBulkProcessorHandler.java:
##########
@@ -161,7 +160,7 @@ private BulkProcessor initBulkProcessor() {
}
protected BiConsumer<BulkRequest, ActionListener<BulkResponse>>
requestConsumer() {
- return (request, bulkListener) ->
elasticConnection.getClient().bulkAsync(request, RequestOptions.DEFAULT,
bulkListener);
+ return (request, bulkListener) ->
elasticConnection.getOldClient().bulkAsync(request, RequestOptions.DEFAULT,
bulkListener);
Review Comment:
Can we add a TODO here and anyplace where we are using the oldClient,
mentioning that it would have to be removed in future with a link to issue
that's preventing us to remove it now ?
--
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]
