rpuch commented on a change in pull request #467:
URL: https://github.com/apache/ignite-3/pull/467#discussion_r756120229
##########
File path:
modules/configuration/src/main/java/org/apache/ignite/internal/configuration/asm/ConfigurationAsmGenerator.java
##########
@@ -2611,7 +2622,13 @@ private BytecodeBlock treatSourceForConstruct(
return codeBlock;
}
-
+
+ private String polyConfTypeIsNotDefinedMessage(Field polymorphicIdField) {
+ return "Polymorphic configuration type is not defined: "
+ + polymorphicIdField.getDeclaringClass().getName()
+ + ". Maybe default value is missing on a @PolymorphicId field
(with hasDefault=true)?";
Review comment:
I actually changed the error message, I just forgot to mention it here.
As it is an internal assertion, I changed it a bit differently to help the one
who hits it debug the code. I'm not sure whether it makes sense to copy-paste
the documentation from an annotation here.
Is this looking better now?
--
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]
