Skip to content

Commit 641f3ac

Browse files
author
Jade Wang
committed
fix(csharp): use case-insensitive comparer when merging connection properties
This fixes a bug where properties passed via Connect() options weren't being found in StatementExecutionStatement because the merged dictionary was using case-sensitive comparison instead of case-insensitive. Without this fix, parameters like ResultDisposition weren't being applied correctly.
1 parent b40b48f commit 641f3ac

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

csharp/src/DatabricksDatabase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public override AdbcConnection Connect(IReadOnlyDictionary<string, string>? opti
4949
? properties
5050
: options
5151
.Concat(properties.Where(x => !options.Keys.Contains(x.Key, StringComparer.OrdinalIgnoreCase)))
52-
.ToDictionary(kvp => kvp.Key, kvp => kvp.Value);
52+
.ToDictionary(kvp => kvp.Key, kvp => kvp.Value, StringComparer.OrdinalIgnoreCase);
5353

5454
// Check protocol parameter to determine which connection type to create
5555
string protocol = "thrift"; // Default to Thrift for backward compatibility

0 commit comments

Comments
 (0)