From 06263257a4c661d7dacbbf188943452c54ffb9cd Mon Sep 17 00:00:00 2001 From: Anas Khan <83116240+anxkhn@users.noreply.github.com> Date: Sun, 5 Jul 2026 17:50:39 +0530 Subject: [PATCH] MINOR: Fix VariantUtil.getByte() Javadoc copied from getShort() 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> --- .../src/main/java/org/apache/parquet/variant/VariantUtil.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/parquet-variant/src/main/java/org/apache/parquet/variant/VariantUtil.java b/parquet-variant/src/main/java/org/apache/parquet/variant/VariantUtil.java index f50a0f3162..ad7165fcfe 100644 --- a/parquet-variant/src/main/java/org/apache/parquet/variant/VariantUtil.java +++ b/parquet-variant/src/main/java/org/apache/parquet/variant/VariantUtil.java @@ -546,9 +546,9 @@ static short getShort(ByteBuffer value) { } /** - * Similar to getLong(), but for the types: Type.BYTE, SHORT. + * Similar to getLong(), but for the type: Type.BYTE. * @param value The Variant value - * @return The short value + * @return The byte value */ static byte getByte(ByteBuffer value) { checkIndex(value.position(), value.limit());