This is an automated email from the ASF dual-hosted git repository.

curth pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-adbc.git


The following commit(s) were added to refs/heads/main by this push:
     new 5465573ec fix(csharp/src/Drivers/Databricks): Fix Lz4 compression 
logic for DatabricksReader (#2690)
5465573ec is described below

commit 5465573ecc8771b39417686e33c5d2a23fcb1d0e
Author: eric-wang-1990 <[email protected]>
AuthorDate: Mon Apr 14 17:57:23 2025 +0000

    fix(csharp/src/Drivers/Databricks): Fix Lz4 compression logic for 
DatabricksReader (#2690)
    
    This PR
    
https://github.com/apache/arrow-adbc/commit/9ba6bdb8bd42e35237dadfae758419846e6c442c
    introduce a regression for DatabricksReader.
    Fix it, also remove the unneeded constructor to prevent any misuse in
    the future.
---
 csharp/src/Drivers/Databricks/DatabricksConnection.cs | 2 +-
 csharp/src/Drivers/Databricks/DatabricksReader.cs     | 5 -----
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/csharp/src/Drivers/Databricks/DatabricksConnection.cs 
b/csharp/src/Drivers/Databricks/DatabricksConnection.cs
index 489a10e97..a6ab5c6ff 100644
--- a/csharp/src/Drivers/Databricks/DatabricksConnection.cs
+++ b/csharp/src/Drivers/Databricks/DatabricksConnection.cs
@@ -66,7 +66,7 @@ namespace Apache.Arrow.Adbc.Drivers.Databricks
             }
             else
             {
-                return new DatabricksReader(databricksStatement, schema);
+                return new DatabricksReader(databricksStatement, schema, 
isLz4Compressed);
             }
         }
 
diff --git a/csharp/src/Drivers/Databricks/DatabricksReader.cs 
b/csharp/src/Drivers/Databricks/DatabricksReader.cs
index 9d8b864ba..56abfbb20 100644
--- a/csharp/src/Drivers/Databricks/DatabricksReader.cs
+++ b/csharp/src/Drivers/Databricks/DatabricksReader.cs
@@ -34,11 +34,6 @@ namespace Apache.Arrow.Adbc.Drivers.Databricks
         IArrowReader? reader;
         bool isLz4Compressed;
 
-        public DatabricksReader(DatabricksStatement statement, Schema schema)
-            : this(statement, schema, false)
-        {
-        }
-
         public DatabricksReader(DatabricksStatement statement, Schema schema, 
bool isLz4Compressed)
         {
             this.statement = statement;

Reply via email to