Skip to content
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

Added option to sture Guids as Blobs #284 #739

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

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