tkalkirill commented on code in PR #1892:
URL: https://github.com/apache/ignite-3/pull/1892#discussion_r1161779526
##########
modules/index/src/main/java/org/apache/ignite/internal/index/IndexBuilder.java:
##########
@@ -133,71 +176,77 @@ private BuildIndexTask(
@Override
public void run() {
- if (!busyLock.enterBusy()) {
- return;
- }
+ completedFuture(null)
+ .thenCompose(unused -> inBusyLock(() -> {
+ // At the time of creating the index, we should have
already waited for the table to be created and its raft of
+ // clients (services) to start for all partitions, so
there should be no errors.
+ RaftGroupService raftGroupService =
table.internalTable().partitionRaftGroupService(partitionId);
+
+ return raftGroupService
+ // We do not check the presence of nodes in
the topology on purpose, so as not to get into races on
+ // rebalancing, it will be more convenient and
reliable for us to wait for a stable topology with a
+ // chosen leader.
Review Comment:
Try fix it.
--
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]
