Skip to content

Commit 6e17502

Browse files
authored
MINOR: Fix VariantUtil.getByte() Javadoc copied from getShort() (apache#3649)
The Javadoc on the package-private VariantUtil.getByte() was copied verbatim from getShort() and never adjusted. It listed SHORT as a supported type and documented an @return of "short value", but the method only accepts the BYTE/INT8 type and returns a byte. Correct the doc to describe Type.BYTE and an @return of "The byte value". No behavioral change. Signed-off-by: Anas Khan <83116240+anxkhn@users.noreply.github.com>
1 parent 09bd186 commit 6e17502

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

parquet-variant/src/main/java/org/apache/parquet/variant/VariantUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -546,9 +546,9 @@ static short getShort(ByteBuffer value) {
546546
}
547547

548548
/**
549-
* Similar to getLong(), but for the types: Type.BYTE, SHORT.
549+
* Similar to getLong(), but for the type: Type.BYTE.
550550
* @param value The Variant value
551-
* @return The short value
551+
* @return The byte value
552552
*/
553553
static byte getByte(ByteBuffer value) {
554554
checkIndex(value.position(), value.limit());

0 commit comments

Comments
 (0)