nizhikovprivate commented on code in PR #12133:
URL: https://github.com/apache/ignite/pull/12133#discussion_r2152726258
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/FileTreeUtils.java:
##########
@@ -70,6 +75,11 @@ public static void removeTmpSnapshotFiles(SnapshotFileTree
sft, boolean err, Ign
* @return Storage path from config for partition.
*/
public static @Nullable String partitionStorage(CacheConfiguration<?, ?>
ccfg, int part) {
+ if (part == INDEX_PARTITION) {
+ if (F.isEmpty(ccfg.getIndexPath()))
Review Comment:
> Maybe we should use !F.isEmpty(...) condition here?
Good catch! Thanks.
> If this comment makes sense I think we should add a test for it.
Actually, can't do it.
This method used to generate storagePath for SnapshotSender.
But, index partition loaded from local snapshot, only.
And result of this method used for loading partitions from remote node.
So, I fixed it, but can't test 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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
