-
Notifications
You must be signed in to change notification settings - Fork 293
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
SqlSpatial #1
Comments
I assume you mean the namespaces System.Data.Spatial and System.Data.Objects.SqlClient (containing SqlSpatialFunctions). No. Spatial is still an existing feature gap between Framework and Core and not one we've targeted as part of this project. We do realize it is still a gap that needs to be filled, though. I don't have any insight for when that might happen. |
I meant SqlGeometry, SqlGeography, SqlHierachy etc from the Microsoft.SqlServer.Types package |
Ah, okay. Same answer, though. We haven't brought anything spatial related into this project. |
😢 |
@dotMorten, the SQL Server team is aware of this feature request and and we will continue discussing with them about bringing Microsoft.SqlServer.Types forward. What can help at this point is to collect concrete data on customers that are blocked from moving their applications because this feature isn't supported in .NET Core, or because it isn't available cross-platform. Also (@David-Engel, feel free to correct me if I am wrong), I believe when using Microsoft.Data.SqlClient on .NET Framework, the functionality is enabled, just like it is in System.Data.SqlClient on .NET Framework. |
@divega I'm unfamiliar with Microsoft.SqlServer.Types but I don't see any dependencies on S.D.SqlClient, so I would assume it works the same with either one, too. |
@dotMorten a similar question came up from the EFCore repo and in order to permit users to work around this problem dotnet/corefx#36678 was added which will allow you to serialize your spatial types in code and pass them through to sql as binary representations directly. That fix should/will be included in this project. |
One of the big problems here is these UDT types doesn't "just work", as the assembly info doesn't match anything client side (assembly name, version and public key must match). otherwise you're left with having to pull out the SqlBinary and parse it yourself. And even after that point you still don't have the same math available to perform the same operations you can perform in SQL server side. It's a really ugly story for a type that's quite literally built into SqlServer. |
but it is. Check my write-up here |
* Removing BinaryFormatter from NetFx * review comments * fix version typo * remove extra line * Reverted SqlException Test * review comments * Review comment * Desrialize * addressing review comments * Fix exception in deserialization (#1) * review comments * add extra line to the end of strings designer * end of line Co-authored-by: jJRahnama <[email protected]> Co-authored-by: Karina Zhou <[email protected]>
Will this repo also include the native source for SqlSpatial, so we can cross compile it and get proper support for these types on Android/iOS/ etc?
The text was updated successfully, but these errors were encountered: