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;