Ewocker commented on a change in pull request #508:
URL: https://github.com/apache/jackrabbit-oak/pull/508#discussion_r823173166



##########
File path: 
oak-run-commons/src/main/java/org/apache/jackrabbit/oak/index/indexer/document/flatfile/FlatFileNodeStoreBuilder.java
##########
@@ -65,6 +65,24 @@
      */
     static final String PROP_THREAD_POOL_SIZE = 
"oak.indexer.dataDumpThreadPoolSize";
 
+    /**
+     * Default value for {@link #PROP_MERGE_THREAD_POOL_SIZE}
+     */
+    static final String DEFAULT_NUMBER_OF_MERGE_TASK_THREADS = "4";
+    /**
+     * System property for specifying number of threads for parallel merge 
when using {@link MultithreadedTraverseWithSortStrategy}
+     */
+    static final String PROP_MERGE_THREAD_POOL_SIZE = 
"oak.indexer.mergeTaskThreadPoolSize";
+
+    /**
+     * Default value for {@link #PROP_MERGE_TASK_BATCH_SIZE}
+     */
+    static final String DEFAULT_NUMBER_OF_FILES_PER_MERGE_TASK = "64";

Review comment:
       I followed the property for DEFAULT_NUMBER_OF_DATA_DUMP_THREADS which is 
also a string
   Should I change it all to int instead?




-- 
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