Skip to content

Commit b909e0a

Browse files
Justas NJustas N
authored andcommitted
Removed old method to register parameterless scalar fuction
1 parent e4aa5f7 commit b909e0a

File tree

2 files changed

+2
-38
lines changed

2 files changed

+2
-38
lines changed

DuckDB.NET.Data/DuckDBConnection.ScalarFunction.cs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -20,16 +20,6 @@ public void RegisterScalarFunction<TResult>(string name, Action<IDuckDBDataWrite
2020
RegisterScalarMethod(name, (_, w, index) => action(w, index), TypeExtensions.GetLogicalType<TResult>(), varargs: false, !isPureFunction);
2121
}
2222

23-
[Obsolete("Prefer using RegisterScalarFunction<TResult>(string name, Action<IDuckDBDataWriter, ulong> action, bool isPureFunction = false)")]
24-
[Experimental("DuckDBNET001")]
25-
#pragma warning disable DuckDBNET001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
26-
public void RegisterScalarFunction<TResult>(string name, Action<IReadOnlyList<IDuckDBDataReader>, IDuckDBDataWriter, ulong> action, bool isPureFunction = false)
27-
{
28-
29-
RegisterScalarMethod(name, action, TypeExtensions.GetLogicalType<TResult>(), varargs: false, !isPureFunction);
30-
}
31-
#pragma warning restore DuckDBNET001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
32-
3323
[Experimental("DuckDBNET001")]
3424
public void RegisterScalarFunction<T, TResult>(string name, Action<IReadOnlyList<IDuckDBDataReader>, IDuckDBDataWriter, ulong> action, bool isPureFunction = true, bool @params = false)
3525
{

DuckDB.NET.Test/ScalarFunctionTests.cs

Lines changed: 2 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -60,32 +60,6 @@ public void RegisterScalarFunctionWithVarargs()
6060
longs.Should().BeEquivalentTo(values);
6161
}
6262

63-
[Fact]
64-
#pragma warning disable CA1041 // Provide ObsoleteAttribute message
65-
[Obsolete]
66-
#pragma warning restore CA1041 // Provide ObsoleteAttribute message
67-
public void RegisterScalarFunctionWithoutParametersObsolete()
68-
{
69-
var values = new List<long>();
70-
Connection.RegisterScalarFunction<long>("my_random_obsolete", (_, writer, rowCount) =>
71-
{
72-
for (ulong index = 0; index < rowCount; index++)
73-
{
74-
var value = Random.Shared.NextInt64();
75-
76-
writer.WriteValue(value, index);
77-
78-
values.Add(value);
79-
}
80-
});
81-
82-
Command.CommandText = "CREATE TABLE big_table_2 AS SELECT (greatest(random(), 0.1) * 10000)::BIGINT i FROM range(100) t(i);";
83-
Command.ExecuteNonQuery();
84-
85-
var longs = Connection.Query<long>("SELECT my_random_obsolete() FROM big_table_2").ToList();
86-
longs.Should().BeEquivalentTo(values);
87-
}
88-
8963
[Fact]
9064
public void RegisterScalarFunctionWithoutParameters()
9165
{
@@ -102,10 +76,10 @@ public void RegisterScalarFunctionWithoutParameters()
10276
}
10377
});
10478

105-
Command.CommandText = "CREATE TABLE big_table_2_1 AS SELECT (greatest(random(), 0.1) * 10000)::BIGINT i FROM range(100) t(i);";
79+
Command.CommandText = "CREATE TABLE big_table_2 AS SELECT (greatest(random(), 0.1) * 10000)::BIGINT i FROM range(100) t(i);";
10680
Command.ExecuteNonQuery();
10781

108-
var longs = Connection.Query<long>("SELECT my_random() FROM big_table_2_1").ToList();
82+
var longs = Connection.Query<long>("SELECT my_random() FROM big_table_2").ToList();
10983
longs.Should().BeEquivalentTo(values);
11084
}
11185

0 commit comments

Comments
 (0)