You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by "SammyVimes (via GitHub)" <gi...@apache.org> on 2023/06/15 06:24:15 UTC

[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2177: IGNITE-19363 Split start of indexes and start of partition raft group nodes

SammyVimes commented on code in PR #2177:
URL: https://github.com/apache/ignite-3/pull/2177#discussion_r1230492955


##########
modules/storage-api/src/main/java/org/apache/ignite/internal/storage/engine/MvTableStorage.java:
##########
@@ -268,7 +268,6 @@ CompletableFuture<Void> finishRebalancePartition(
      * @return Index Storage.
      * @throws StorageException If the given partition does not exist.
      */
-    // TODO: IGNITE-19112 Change or get rid of

Review Comment:
   Because we need this method, it shouldn't be removed. Also, in the ticket there is no mention of the `getIndex` method



-- 
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-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org