nizhikov commented on code in PR #12081:
URL: https://github.com/apache/ignite/pull/12081#discussion_r2114589561
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/NodeFileTree.java:
##########
@@ -560,7 +573,7 @@ public List<File> existingTmpCacheStorages() {
* @return Cache configuration file with respect to {@link
CacheConfiguration#getGroupName} value.
*/
public File cacheConfigurationFile(CacheConfiguration<?, ?> ccfg) {
- return new File(cacheStorage(ccfg), ccfg.getGroupName() == null
+ return new File(cacheStorages(ccfg)[0], ccfg.getGroupName() == null
Review Comment:
Can you, please, be more specific.
`nodeStorage` is root for node-specific files.
cache configuration file must be stored inside `cacheStorage` like we do now.
First directory will point to the current cache directory for the case when
`CacheConfiguration#storagePath == null`. Looks like we don't want to change
this.
Please, take a look at `NodeFileTree` javadoc explaining all variables.
--
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
