chia7712 commented on a change in pull request #10193:
URL: https://github.com/apache/kafka/pull/10193#discussion_r586082468
##########
File path:
clients/src/main/java/org/apache/kafka/common/protocol/types/Type.java
##########
@@ -320,7 +320,7 @@ public Long validate(Object item) {
if (item instanceof Long)
return (Long) item;
else
- throw new SchemaException(item + " is not a Long.");
+ throw new SchemaException(item + " is not an a Long (encoding
an unsigned integer).");
Review comment:
> There is a typo here, "an a".
My bad :(
> Also, I think it reads a bit weird. Not clear that "encoding an unsigned
integer" in brackets means when reading the message.
I will update it in #10248 @ijuma thanks for your reviews
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
