[GitHub] [jackrabbit-oak] gnaresh19 commented on a diff in pull request #715: OAK-9790 - Implement parallel indexing for speeding up oak run indexing command

[GitBox]

gnaresh19 commented on code in PR #715:
URL: https://github.com/apache/jackrabbit-oak/pull/715#discussion_r978985478


##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/index/ElasticBulkProcessorHandler.java:
##########
@@ -151,6 +153,7 @@ private BulkProcessor initBulkProcessor() {
         return BulkProcessor.builder(requestConsumer(),
                 new OakBulkProcessorListener(), this.indexName + 
"-bulk-processor")
                 .setBulkActions(indexDefinition.bulkActions)
+                .setConcurrentRequests(BULK_PROCESSOR_CONCURRENCY)

Review Comment:
   @amit-jain : Can the concurrentRequests property be moved to index 
definition, similar to other properties like bulkSize, which would enable the 
caller to configure appropriate concurrentRequests in elastic index definition.



-- 
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...@jackrabbit.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org