Azure / Azure/azure-functions-sql-extension

Deserialization into POCO

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
enhancement P2
Dominant language
C#
Stars
130
Forks
71
Avg merge
4d 8h
Merged PRs (30d)
4

Description

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 build up a dictionary mapping from column name to column value. We then serialize that dictionary into a JSON string, and then deserialize the dictionary into the POCO type. This is a pretty round-about and inefficient way to do it, so it might be worth looking into alternatives, such as building our own deserializer that uses reflection to map column names to property names.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.