fix: check if handle has been initialized before closing (#1554)

wForget · web-flow · commit 522410851814 · 2025-03-20T15:28:42.000-06:00
* fix: check if handle has been initialized before closing

* address comments
diff --git a/common/src/main/java/org/apache/comet/parquet/NativeBatchReader.java b/common/src/main/java/org/apache/comet/parquet/NativeBatchReader.java
@@ -459,7 +459,10 @@ public void close() throws IOException {
       importer = null;
     }
     nativeUtil.close();
-    Native.closeRecordBatchReader(this.handle);
+    if (this.handle > 0) {
+      Native.closeRecordBatchReader(this.handle);
+      this.handle = 0;
+    }
   }
 
   @SuppressWarnings("deprecation")

Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,10 @@ public void close() throws IOException {`
`459`	`459`	`importer = null;`
`460`	`460`	`}`
`461`	`461`	`nativeUtil.close();`
`462`		`- Native.closeRecordBatchReader(this.handle);`
	`462`	`+ if (this.handle > 0) {`
	`463`	`+ Native.closeRecordBatchReader(this.handle);`
	`464`	`+ this.handle = 0;`
	`465`	`+ }`
`463`	`466`	`}`
`464`	`467`
`465`	`468`	`@SuppressWarnings("deprecation")`