(parquet-mr) branch master updated: PARQUET-2391: Remove unnecessary unboxing (#1214)

[fokko]

This is an automated email from the ASF dual-hosted git repository.

fokko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-mr.git


The following commit(s) were added to refs/heads/master by this push:
     new 2de893ffd PARQUET-2391: Remove unnecessary unboxing (#1214)
2de893ffd is described below

commit 2de893ffd2b4b2ed55f8edb68458f01b1b0a1028
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Mon Dec 4 08:18:42 2023 +0100

    PARQUET-2391: Remove unnecessary unboxing (#1214)
---
 parquet-column/src/main/java/org/apache/parquet/schema/GroupType.java | 2 +-
 .../main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java    | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/parquet-column/src/main/java/org/apache/parquet/schema/GroupType.java 
b/parquet-column/src/main/java/org/apache/parquet/schema/GroupType.java
index 4e2df7a40..2763b8553 100644
--- a/parquet-column/src/main/java/org/apache/parquet/schema/GroupType.java
+++ b/parquet-column/src/main/java/org/apache/parquet/schema/GroupType.java
@@ -175,7 +175,7 @@ public class GroupType extends Type {
     if (i == null) {
       throw new InvalidRecordException(name + " not found in " + this);
     }
-    return i.intValue();
+    return i;
   }
 
   /**
diff --git 
a/parquet-column/src/main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java
 
b/parquet-column/src/main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java
index a526c3c26..85d68c182 100644
--- 
a/parquet-column/src/main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java
+++ 
b/parquet-column/src/main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java
@@ -80,7 +80,7 @@ public abstract class LogicalTypeAnnotation {
         if (params.size() != 2) {
           throw new RuntimeException("Expecting 2 parameters for decimal 
logical type, got " + params.size());
         }
-        return decimalType(Integer.valueOf(params.get(1)), 
Integer.valueOf(params.get(0)));
+        return decimalType(Integer.parseInt(params.get(1)), 
Integer.parseInt(params.get(0)));
       }
     },
     DATE {
@@ -114,7 +114,7 @@ public abstract class LogicalTypeAnnotation {
         if (params.size() != 2) {
           throw new RuntimeException("Expecting 2 parameters for integer 
logical type, got " + params.size());
         }
-        return intType(Integer.valueOf(params.get(0)), 
Boolean.parseBoolean(params.get(1)));
+        return intType(Integer.parseInt(params.get(0)), 
Boolean.parseBoolean(params.get(1)));
       }
     },
     JSON {