nielsbasjes commented on code in PR #2210:
URL: https://github.com/apache/avro/pull/2210#discussion_r1185187952
##########
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:
I would say yes to that and we should have at least one test that shows the
combination of the two work.
I believe the Java versions should be able to read a file with this and also
be able to write the same data to a new file with the exact same data in it
(i.e. also needing the long).
--
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]
