JAkutenshi commented on code in PR #5300:
URL: https://github.com/apache/ignite-3/pull/5300#discussion_r1976481708
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -2814,7 +2835,11 @@ private CompletableFuture<Void>
stopPartition(TablePartitionId tablePartitionId,
CompletableFuture<Boolean> stopReplicaFuture;
try {
- stopReplicaFuture = replicaMgr.stopReplica(tablePartitionId);
+ // In case of colocation there shouldn't be any table replica and
thus it shouldn't be stopped. Moreover the excessive replica
Review Comment:
1. Huh, yes, I agree, but why I left it as is: because it was such before me
and there might be a resource release protection (who knows why replica wasn't
stopped, but there still shouldn't be the raft node).
2. We have a raft-node per zone partition, that starts in
`PartitionReplicaLifecycleManager#createZonePartitionReplicationNode`. We
shouldn't have the node with table and I want to play it safe there.
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
