clesaec commented on code in PR #2210:
URL: https://github.com/apache/avro/pull/2210#discussion_r1185073866
##########
lang/java/avro/src/main/java/org/apache/avro/io/BinaryDecoder.java:
##########
@@ -329,9 +329,10 @@ public void skipString() throws IOException {
@Override
public ByteBuffer readBytes(ByteBuffer old) throws IOException {
- int length = readInt();
+ long length = readLong();
Review Comment:
Should this change also be done in
[BinaryEncoder](https://github.com/apache/avro/blob/master/lang/java/avro/src/main/java/org/apache/avro/io/BinaryEncoder.java#L66)
to replace writeInt by writelong ?
(At least, unit test that chain encoder => decoder, to ensure WriteInt =>
ReadLong is 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]
