Missing dispose of RowGroupMetaData in RowGroupReader?

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+    "cquery.cacheDirectory": "${workspaceFolder}/.vscode/cquery_cached_index/"
+}

cpp/RowGroupMetaData.cpp

@@ -8,6 +8,11 @@ using namespace parquet;
 
 extern "C"
 {
+	PARQUETSHARP_EXPORT void RowGroupMetaData_Free(const RowGroupMetaData* row_group_meta_data)
+	{
+		delete row_group_meta_data;
+	}
+
 	PARQUETSHARP_EXPORT ExceptionInfo* RowGroupMetaData_Get_Column_Chunk_Meta_Data(const RowGroupMetaData* row_group_meta_data, int i, ColumnChunkMetaData** column_chunk_meta_data)
 	{
 		TRYCATCH(*column_chunk_meta_data = row_group_meta_data->ColumnChunk(i).release();)

cpp/SchemaDescriptor.cpp

@@ -8,6 +8,11 @@ using namespace parquet;
 
 extern "C"
 {
+	PARQUETSHARP_EXPORT void SchemaDescriptor_Free(const SchemaDescriptor* descriptor)
+	{
+		delete descriptor;
+	}
+
 	PARQUETSHARP_EXPORT ExceptionInfo* SchemaDescriptor_Column(const SchemaDescriptor* descriptor, int i, const ColumnDescriptor** column_descriptor)
 	{
 		TRYCATCH(*column_descriptor = descriptor->Column(i);)

csharp/RowGroupMetaData.cs

@@ -3,11 +3,17 @@
 
 namespace ParquetSharp
 {
-    public sealed class RowGroupMetaData
+    public sealed class RowGroupMetaData : IDisposable
     {
         internal RowGroupMetaData(IntPtr handle)
         {
-            _handle = handle;
+            _handle = new ParquetHandle(handle, RowGroupMetaData_Free);
+        }
+
+        public void Dispose()
+        {
+            _schema?.Dispose();
+            _handle.Dispose();
         }
 
         public int NumColumns => ExceptionInfo.Return<int>(_handle, RowGroupMetaData_Num_Columns);
@@ -35,7 +41,10 @@ public ColumnChunkMetaData GetColumnChunkMetaData(int i)
         [DllImport(ParquetDll.Name)]
         private static extern IntPtr RowGroupMetaData_Total_Byte_Size(IntPtr rowGroupMetaData, out long totalByteSize);
 
-        private readonly IntPtr _handle;
+        [DllImport(ParquetDll.Name)]
+        private static extern void RowGroupMetaData_Free(IntPtr rowGroupMetaData);
+
+        private readonly ParquetHandle _handle;
         private SchemaDescriptor? _schema;
     }
 }

csharp/RowGroupReader.cs

@@ -13,6 +13,7 @@ internal RowGroupReader(IntPtr handle, ParquetFileReader parquetFileReader)
 
         public void Dispose()
         {
+            _metaData?.Dispose();
             _handle.Dispose();
         }
 

csharp/SchemaDescriptor.cs

@@ -4,11 +4,16 @@
 
 namespace ParquetSharp
 {
-    public sealed class SchemaDescriptor
+    public sealed class SchemaDescriptor : IDisposable
     {
         internal SchemaDescriptor(IntPtr schemaDescriptorHandle)
         {
-            _handle = schemaDescriptorHandle;
+            _handle = new ParquetHandle(schemaDescriptorHandle, SchemaDescriptor_Free);
+        }
+
+        public void Dispose()
+        {
+            _handle.Dispose();
         }
 
         public GroupNode GroupNode => (GroupNode) (Node.Create(ExceptionInfo.Return<IntPtr>(_handle, SchemaDescriptor_Group_Node)) ?? throw new InvalidOperationException());
@@ -38,6 +43,9 @@ public Node ColumnRoot(int i)
             return Node.Create(ExceptionInfo.Return<int, IntPtr>(_handle, i, SchemaDescriptor_Get_Column_Root)) ?? throw new InvalidOperationException();
         }
 
+        [DllImport(ParquetDll.Name)]
+        private static extern void SchemaDescriptor_Free(IntPtr descriptor);
+
         [DllImport(ParquetDll.Name)]
         private static extern IntPtr SchemaDescriptor_Column(IntPtr descriptor, int i, out IntPtr columnDescriptor);
 
@@ -62,6 +70,6 @@ public Node ColumnRoot(int i)
         [DllImport(ParquetDll.Name)]
         private static extern IntPtr SchemaDescriptor_Schema_Root(IntPtr descriptor, out IntPtr schemaRoot);
 
-        private readonly IntPtr _handle;
+        private readonly ParquetHandle _handle;
     }
 }