kirktrue commented on code in PR #18466:
URL: https://github.com/apache/kafka/pull/18466#discussion_r1909471600
##########
clients/src/main/java/org/apache/kafka/common/serialization/Deserializer.java:
##########
@@ -63,7 +63,15 @@ default T deserialize(String topic, Headers headers, byte[]
data) {
}
/**
- * Deserialize a record value from a ByteBuffer into a value or object.
+ * Deserialize a record value from a {@link ByteBuffer} into a value or
object.
+ *
+ * <p>If {@code ByteBufferDeserializer} is used by an application, the
application code cannot make any assumptions
+ * about the returned {@link ByteBuffer} like the position, limit,
capacity, etc., or if it is backed by
+ * {@link ByteBuffer#hasArray() an array or not}.
+ *
+ * <p>Similarly, if this method is overwritten, the implementation cannot
make any assumptions about the
Review Comment:
```suggestion
* <p>Similarly, if this method is overridden, the implementation cannot
make any assumptions about the
```
--
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]
