Fix the use of raw integers for Arrow status codes

Author: adamreeve

cpp/Enums.cpp

@@ -97,6 +97,10 @@ namespace
 		static_assert(::arrow::TimeUnit::type::MILLI == 1);
 		static_assert(::arrow::TimeUnit::type::MICRO == 2);
 		static_assert(::arrow::TimeUnit::type::NANO == 3);
-	}
 
+		// Arrow StatusCode values that should be kept up to date with ArrowStatusCode.cs
+		static_assert((int) ::arrow::StatusCode::OutOfMemory == 1);
+		static_assert((int) ::arrow::StatusCode::IOError == 5);
+		static_assert((int) ::arrow::StatusCode::UnknownError == 9);
+	}
 }

csharp/ArrowStatusCode.cs

@@ -0,0 +1,12 @@
+namespace ParquetSharp
+{
+    /// <summary>
+    /// Subset of Arrow StatusCode enums used from ParquetSharp.
+    /// </summary>
+    internal enum ArrowStatusCode
+    {
+        OutOfMemory = 1,
+        IOError = 5,
+        UnknownError = 9,
+    }
+}

csharp/IO/ManagedRandomAccessFile.cs

@@ -168,16 +168,16 @@ private byte HandleException(Exception error, out string? exception)
             if (error is OutOfMemoryException)
             {
                 exception = _exceptionMessage = null;
-                return 1;
+                return (byte) ArrowStatusCode.OutOfMemory;
             }
             if (error is IOException)
             {
                 exception = _exceptionMessage = error.ToString();
-                return 5;
+                return (byte) ArrowStatusCode.IOError;
             }
 
             exception = _exceptionMessage = error.ToString();
-            return 9;
+            return (byte) ArrowStatusCode.UnknownError;
         }
 
         [DllImport(ParquetDll.Name)]