sashapolo commented on code in PR #6478:
URL: https://github.com/apache/ignite-3/pull/6478#discussion_r2298128066
##########
modules/cluster-management/src/main/java/org/apache/ignite/internal/cluster/management/ClusterInitializer.java:
##########
@@ -225,7 +225,7 @@ public CompletableFuture<Void> initCluster(
e = e.getCause();
}
- LOG.info("Initialization failed
[reason={}]", e, e.getMessage());
+ LOG.warn("Initialization failed
[reason={}]", e.getMessage());
Review Comment:
This line is about printing the stacktrace to the logs, not to the end user.
The logic here should be the following: if this error is caused by user actions
(i.e. invalid user input) then no stacktrace is needed, otherwise, if this is
caused by an internal server error, we definitely need a stacktrace
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
