AMashenkov commented on code in PR #6687:
URL: https://github.com/apache/ignite-3/pull/6687#discussion_r2413349865
##########
modules/marshaller-common/src/main/java/org/apache/ignite/internal/marshaller/Marshaller.java:
##########
@@ -56,15 +56,15 @@ static Marshaller createMarshaller(
}
if (mapper.targetType().isPrimitive()) {
- throw new IllegalArgumentException("Mappers for primitive types
are not supported: " + mapper.targetType());
+ throw new MarshallerException("Mappers for primitive types are not
supported: " + mapper.targetType());
}
if (mapper instanceof OneColumnMapper) {
return simpleMarshaller(cols, (OneColumnMapper<?>) mapper);
} else if (mapper instanceof PojoMapper) {
return pojoMarshaller(cols, (PojoMapper<?>) mapper,
requireAllFields, allowUnmappedFields);
} else {
- throw new IllegalArgumentException("Mapper of unsupported type: "
+ mapper.getClass());
+ throw new MarshallerException("Mapper of unsupported type: " +
mapper.getClass());
Review Comment:
It would be nice to use `IgniteStringFormatter.format` here an below instead
of "Ignite 2 style".
--
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]
