zstan commented on a change in pull request #8837:
URL: https://github.com/apache/ignite/pull/8837#discussion_r601066370
##########
File path:
modules/control-utility/src/test/java/org/apache/ignite/util/GridCommandHandlerIndexForceRebuildTest.java
##########
@@ -87,10 +93,11 @@
private static final int LAST_NODE_NUM = GRIDS_NUM - 1;
/**
- * Set containing names of caches for which index rebuild should be
blocked.
- * See {@link BlockingIndexing}.
+ * Map for blocking index rebuilds in a {@link BlockingIndexing}.
+ * To stop blocking, need to delete the entry.
+ * Mapping: cache name -> future start blocking rebuilding indexes.
*/
- private static Set<String> cacheNamesBlockedIdxRebuild = new
GridConcurrentHashSet<>();
+ private static final Map<String, GridFutureAdapter<Void>> blockRebuildIdx
= new ConcurrentHashMap<>();
Review comment:
1. what is the purpose for CHM here?
2. why do we need map here? I can`t find goal for future usage here.
--
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:
[email protected]
