fix: Make all JNI types support idempotent close (#3752)

Spark/Iceberg ends up calling close multiple times

Signed-off-by: Andrew Duffy <andrew@a10y.dev>

---------

Signed-off-by: Andrew Duffy <andrew@a10y.dev>

Author: a10y

java/vortex-jni/src/main/java/dev/vortex/jni/JNIArray.java

@@ -153,6 +153,10 @@ public byte[] getBinary(int index) {
 
     @Override
     public void close() {
+        if (pointer.isEmpty()) {
+            return;
+        }
+
         NativeArrayMethods.free(pointer.getAsLong());
         pointer = OptionalLong.empty();
     }

java/vortex-jni/src/main/java/dev/vortex/jni/JNIArrayIterator.java

@@ -51,6 +51,10 @@ public DType getDataType() {
 
     @Override
     public void close() {
+        if (pointer.isEmpty()) {
+            return;
+        }
+
         NativeArrayIteratorMethods.free(pointer.getAsLong());
         pointer = OptionalLong.empty();
         next = Optional.empty();

java/vortex-jni/src/main/java/dev/vortex/jni/JNIDType.java

@@ -101,7 +101,7 @@ public byte getScale() {
 
     @Override
     public void close() {
-        if (shouldFree) {
+        if (shouldFree && pointer.isPresent()) {
             NativeDTypeMethods.free(pointer.getAsLong());
             pointer = OptionalLong.empty();
         }

java/vortex-jni/src/main/java/dev/vortex/jni/JNIFile.java

@@ -60,6 +60,9 @@ public ArrayIterator newScan(ScanOptions options) {
 
     @Override
     public void close() {
+        if (pointer.isEmpty()) {
+            return;
+        }
         NativeFileMethods.close(pointer.getAsLong());
         pointer = OptionalLong.empty();
     }