Merge pull request apache#7 from shangxinli/encr_commit_50a29d0

Fix bug: Optional field in RowGroup is treated as always showing up
chenjunjiedada · May 3, 2019 · e557b86 · e557b86
2 parents 35a8c8a + 89e5478
commit e557b86
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/metadata/BlockMetaData.java b/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/metadata/BlockMetaData.java
@@ -46,10 +46,17 @@ public BlockMetaData() {
 
   // Reader side (get parameters from RowGroup structure)
   public BlockMetaData(long fileOffset, long totalCompressedSize) {
-    this.startingPosition = fileOffset;
-    startingPositionSet = true;
-    this.totalCompressedSize = totalCompressedSize;
-    totalCompressedSizeSet = true;
+    // fileOffset is optional, 0 means fileOffset is absent
+    if (fileOffset > 0) {
+      this.startingPosition = fileOffset;
+      startingPositionSet = true;
+    }
+
+    // totalCompressedSize is optional, 0 means totalCompressedSize is absent
+    if (totalCompressedSize > 0) {
+      this.totalCompressedSize = totalCompressedSize;
+      totalCompressedSizeSet = true;
+    }
   }