-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Discussion around Dapper vNext #34
Comments
My quick analysis of options:
Or maybe there is a third option for F# accessibility to C# source generators that I’m missing. |
Hi Jordan, thanks for the info and analysis. I think we can stick with no.2 for some time (at least until Dapper vNext is at a reasonable state, hotfixed, etc...) and then we can work on Dapper.FSharp v3.0.0. I wonder what it offers from the developer's point of view. 🤔 I mean... the current state is great and covers most of the scenarios you'd normally need. I could not find much of this information in DapperLib/Dapper#1637. So maybe we'll just sit, wait and watch Nick for some time to see where is it all heading? |
I’m guessing that the purpose would be to replace the reflection based property loader with generated loaders. So the upside will be slightly faster performance for C# users at the expense of compatibility with F#. |
It looks like Dapper vNext is going to heavily incorporate C# source generators which will break compatibility with F#.
It’s probably 3-6 months out, so this would be a good time to discuss impact and possible strategies.
See this tweet
There is also an issue on the Dapper Issues page entitled “Project state?” that discusses it.
The text was updated successfully, but these errors were encountered: