shangxinli commented on code in PR #960: URL: https://github.com/apache/parquet-mr/pull/960#discussion_r950906824
########## parquet-common/src/main/java/org/apache/parquet/bytes/ByteBufferInputStream.java: ########## @@ -138,6 +134,18 @@ public int read(byte[] b, int off, int len) throws IOException { return delegate.read(b, off, len); } + public int read(byte[] b) throws IOException { + return read(b, 0, b.length); + } + + public void readFully(byte b[]) throws IOException { Review Comment: Why we need to be different between line 137 ' byte[] b' and 141 ' byte b[]'? -- 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: dev-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org