GH-765: Do not close/free imported BaseStruct objects

Author: pepijnve

c/src/main/java/org/apache/arrow/c/ArrayImporter.java

@@ -58,7 +58,6 @@ void importArray(ArrowArray src) {
     ArrowArray ownedArray = ArrowArray.allocateNew(allocator);
     ownedArray.save(snapshot);
     src.markReleased();
-    src.close();
 
     recursionLevel = 0;
 

c/src/main/java/org/apache/arrow/c/ArrowArrayStreamReader.java

@@ -44,7 +44,6 @@ final class ArrowArrayStreamReader extends ArrowReader {
     this.ownedStream = ArrowArrayStream.allocateNew(allocator);
     this.ownedStream.save(snapshot);
     stream.markReleased();
-    stream.close();
   }
 
   @Override

c/src/main/java/org/apache/arrow/c/Data.java

@@ -236,6 +236,7 @@ public static void exportArrayStream(
    *
    * <p>The given ArrowSchema struct is released (as per the C data interface specification), even
    * if this function fails.
+   * <p>The given ArrowSchema struct is not closed (freed). This is the responsibility of the caller.
    *
    * @param allocator Buffer allocator for allocating dictionary vectors
    * @param schema C data interface struct representing the field [inout]
@@ -250,7 +251,6 @@ public static Field importField(
       return importer.importField(schema, provider);
     } finally {
       schema.release();
-      schema.close();
     }
   }
 
@@ -279,8 +279,9 @@ public static Schema importSchema(
   /**
    * Import Java vector from the C data interface.
    *
-   * <p>The ArrowArray struct has its contents moved (as per the C data interface specification) to
-   * a private object held alive by the resulting array.
+   * <p>On successful completion, the ArrowArray struct will have been moved (as per the C data interface specification)
+   * to a private object held alive by the resulting array. This means the ArrowArray will have been released, but not
+   * closed (freed). Closing the ArrowArray is the responsibility of the caller.
    *
    * @param allocator Buffer allocator
    * @param array C data interface struct holding the array data
@@ -401,6 +402,10 @@ public static VectorSchemaRoot importVectorSchemaRoot(
   /**
    * Import an ArrowArrayStream as an {@link ArrowReader}.
    *
+   * <p>On successful completion, the ArrowArrayStream struct will have been moved (as per the C data interface specification)
+   *  to a private object held alive by the resulting ArrowReader. This means the ArrowArrayStream will have been
+   *  released, but not closed (freed). Closing the ArrowArrayStream is the responsibility of the caller.
+   *
    * @param allocator Buffer allocator for allocating the output data.
    * @param stream C stream interface struct to import.
    * @return Imported reader