sophiatev
sophiatev
The way a user currently passes parameters to the SQL query included in their input binding is via the [`Parameters`](https://github.com/Azure/azure-functions-sql-extension/blob/dev/src/SqlBinding/SqlAttribute.cs#L61) parameter of the SqlAttribute. This isn't a great way to...
It's unsafe to rely on a user calling [`DisposeAsync`](https://github.com/Azure/azure-functions-sql-extension/blob/a2a8da1bc448321f1adfe64c7c7ffdc54f6c3238/src/SqlAsyncEnumerable.cs#L77) once they're done using the enumerator exposed by `SqlAsyncEnumerable`. If they do not call it, the `SqlConnection` and `SqlDataReader` will remain...
We currently use `SqlDataAdapter` for both the input binding in the `SqlConverters` [class ](https://github.com/Azure/azure-functions-sql-extension/blob/dev/src/SqlBinding/SqlConverters.cs#L111) and for the output binding in the `SqlAsyncCollector` [class](https://github.com/Azure/azure-functions-sql-extension/blob/dev/src/SqlBinding/SqlAsyncCollector.cs#L103). The reason we do so is because...
Currently, whenever we have to deserialize the SQL rows from the user's table into a POCO (like [here](https://github.com/Azure/azure-functions-sql-extension/blob/dev/src/SqlBinding/SqlAsyncEnumerable.cs#L117) for the input binding and [here](https://github.com/Azure/azure-functions-sql-extension/blob/dev/src/SqlBinding/TriggerBinding/SqlTableWatcher.cs#L735) for the trigger binding), we first...