Skip to content

Conversation

gentledepp
Copy link

Hi!

This PR adds the ability to store guids as blobs in the database and fixes issue #284

I added unit tests for both, async and sync connection.

This feature is very important for interoperability with EF Core, which is based on Microsoft.Data.Sqlite. The latter stores guids as blobs always and does not, and will never support any other option, as can be read in the following issue:
aspnet/Microsoft.Data.Sqlite#273

Also: I found a nice article which can help to look at the blob-guids in the database - as they are obviously not that readable anymore...
https://neosmart.net/blog/2018/converting-a-binary-blob-guid-to-text-in-sql/

@gentledepp
Copy link
Author

I need some help please:
Clicking on the Details link shows me a 404 not found... 👎

@gentledepp
Copy link
Author

@praeclarum - what am I doing wrong?

@gentledepp
Copy link
Author

It seems that the build is not configured anymore.
When clicking the Build Status link on the main page (readme), I also get a 404 - so how to proceed?

Also, I found a (in my opinion) major issue with the current source code version. See #740

alexander.marek added 2 commits August 8, 2018 15:59
…ing the connection at all leads to a NullReferenceException (as no Entry exists that can be Closed())
@gentledepp
Copy link
Author

Hi!
Any chance this will ever be merged?? Supporting guids as blobs is very important for interoperability with other libraries that use Microsoft.Data.Sqlite which exclusively formats guids as blob (EF Core, Dotmim.Sync, etc.)

…nd set AssemblyVersion to 1.1.0.0 also for compatibility
@ghost
Copy link

ghost commented Aug 26, 2022

@praeclarum any chance this will ever get merged back, one of the databases I am working on uses BLOBs to store GUIDs, so having this would make reading the GUIDs valid for that column, right now I have to manually parse that entire table into the object representation because of the BLOBs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants