ivanzlenko commented on code in PR #6549:
URL: https://github.com/apache/ignite-3/pull/6549#discussion_r2348183147
##########
modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java:
##########
@@ -1693,20 +1695,26 @@ private RuntimeException handleStartException(Throwable
e) {
var igniteException = new IgniteException(extractCodeFrom(e), errMsg,
e);
- // We log the exception as soon as possible to minimize the
probability that it gets lost due to something like an OOM later.
- LOG.error(errMsg, igniteException);
+ Throwable rootEx = unwrapRootCause(e);
+ if (rootEx instanceof InvalidNodeConfigurationException) {
Review Comment:
In that case please fix it. The intention of this ticket was introducing a
way to separate some kind of errors, and not to change the behavior of the
product.
--
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]
