sashapolo commented on code in PR #6209:
URL: https://github.com/apache/ignite-3/pull/6209#discussion_r2191694010
##########
modules/runner/src/main/java/org/apache/ignite/internal/configuration/storage/LocalFileConfigurationStorage.java:
##########
@@ -365,6 +371,13 @@ private void checkAndRestoreConfigFile() {
} catch (IOException e) {
throw new NodeConfigWriteException("Failed to restore config
file.", e);
}
+ } else if (!Files.isWritable(configPath)) {
Review Comment:
Not related to this PR, but it would be cool and more consistent if you
could replace `configPath.toFile()` calls above with Java NIO API
--
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
