rpuch commented on code in PR #7598:
URL: https://github.com/apache/ignite-3/pull/7598#discussion_r2832422089
##########
modules/partition-replicator/src/main/java/org/apache/ignite/internal/partition/replicator/schemacompat/CompatValidationResult.java:
##########
@@ -154,4 +156,25 @@ public String details() {
return details;
}
+
+ /**
+ * Returns error message corresponding to validation failure. Should only
be called for a failed validation result, otherwise an
+ * assertion error may be thrown.
+ */
+ public String validationFailedMessage() {
Review Comment:
Different exceptions can be constructed with this error message. Also, the
Raft safe time validator doesn't need the exception itself, just the error
message to construct a response message. So I think error message is more
suitable here
--
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]
