radai-rosenblatt commented on a change in pull request #1550:
URL: https://github.com/apache/avro/pull/1550#discussion_r807625624
##########
File path: lang/java/avro/src/main/java/org/apache/avro/Schema.java
##########
@@ -1585,6 +1585,13 @@ private static String validateName(String name) {
return name;
}
+ private static <T> T requireNotNull(T input, String name) {
Review comment:
this would throw an NPE and not an IllegalArgumentException. to most
developers an NPE looks like a bug in the library rather than failed input
validation.
but ok
--
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]
