Fix checkstyle for main.

apache · Dec 12, 2024 · c220d50 · c220d50
1 parent e962312
commit c220d50
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/core/src/main/java/org/apache/iceberg/variants/ShreddedObject.java b/core/src/main/java/org/apache/iceberg/variants/ShreddedObject.java
@@ -117,7 +117,7 @@ private SerializationState(
       this.fieldIdSize = VariantUtil.sizeOf(metadata.dictionarySize());
       this.shreddedFields = shreddedFields;
 
-      int dataSize = 0;
+      int totalDataSize = 0;
       // get the unshredded field names and values as byte buffers
       ImmutableMap.Builder<String, ByteBuffer> unshreddedBuilder = ImmutableMap.builder();
       if (unshredded != null) {
@@ -128,7 +128,7 @@ private SerializationState(
           if (!replaced) {
             ByteBuffer value = unshredded.sliceValue(field.second());
             unshreddedBuilder.put(name, value);
-            dataSize += value.remaining();
+            totalDataSize += value.remaining();
           }
         }
       }
@@ -140,12 +140,12 @@ private SerializationState(
       this.isLarge = numElements > 0xFF;
 
       for (VariantValue value : shreddedFields.values()) {
-        dataSize += value.sizeInBytes();
+        totalDataSize += value.sizeInBytes();
       }
 
-      this.dataSize = dataSize;
+      this.dataSize = totalDataSize;
       // offset size is the size needed to store the length of the data section
-      this.offsetSize = VariantUtil.sizeOf(dataSize);
+      this.offsetSize = VariantUtil.sizeOf(totalDataSize);
     }
 
     private int size() {

diff --git a/core/src/main/java/org/apache/iceberg/variants/Variants.java b/core/src/main/java/org/apache/iceberg/variants/Variants.java
@@ -25,6 +25,8 @@
 import org.apache.iceberg.util.DateTimeUtil;
 
 public class Variants {
+  private Variants() {}
+
   enum LogicalType {
     NULL,
     BOOLEAN,