[GitHub] [ignite] denis-chudov commented on a change in pull request #6687: IGNITE-11979 Add ability to set default parallelism of rebuild indexes in ignite system properties

[GitBox <gi...@apache.org>]

denis-chudov commented on a change in pull request #6687: IGNITE-11979 Add ability to set default parallelism of rebuild indexes in ignite system properties
URL: https://github.com/apache/ignite/pull/6687#discussion_r306214512
 
 

 ##########
 File path: modules/core/src/main/java/org/apache/ignite/internal/processors/query/schema/SchemaIndexCacheVisitorImpl.java
 ##########
 @@ -68,6 +69,15 @@
     /** Whether to stop the process. */
     private volatile boolean stop;
 
+    static {
+        int parallelism = IgniteSystemProperties.getInteger(INDEX_REBUILDING_PARALLELISM, 0);
+
+        if (parallelism > 0)
+            DFLT_PARALLELISM = Math.min(parallelism, Runtime.getRuntime().availableProcessors());
 
 Review comment:
   There is a description in javadoc for INDEX_REBUILDING_PARALLELISM, and overriding is mentioned there. Do you suggest to add logging for this overriding?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services