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

Add a section to cover that native database-generated concurrency tokens aren't supported for SQLite #4827

Closed
guardrex opened this issue Oct 6, 2024 · 1 comment · Fixed by #4829
Assignees
Milestone

Comments

@guardrex
Copy link
Contributor

guardrex commented Oct 6, 2024

Type of issue

Missing information

Description

See if I've made a mistake in my reasoning on this, but this article might include a short section on this subject.

I was investigating why concurrency wasn't working OOB with our Blazor-EF Core sample app, and I found the following remark at Handling Concurrency Conflicts: Native database-generated concurrency tokens (emphasis added):

The rowversion type shown above is a SQL Server-specific feature; the details on setting up an automatically-updating concurrency token differ across databases, and some databases don't support these at all (e.g. SQLite).

Indeed! Our Blazor-EF Core sample app (built by Jeremy Likness) that uses SQLite calls IsRowVersion to use automatic concurrency tokens.

In our Blazor article that accompanies the sample app, I'll call this out, which is tracked by dotnet/AspNetCore.Docs#33799 and resolved by dotnet/AspNetCore.Docs#33800.

My ask here is if you would like to add a short section to this article on SQLite limitations that native database-generated concurrency tokens aren't supported for SQLite? Just in case devs don't see the concurrency article that I mentioned above, they might catch this limitation in this special limitations-focused article.

Page URL

https://learn.microsoft.com/en-us/ef/core/providers/sqlite/limitations

Content source URL

https://github.com/dotnet/EntityFramework.Docs/blob/main/entity-framework/core/providers/sqlite/limitations.md

Document Version Independent Id

545bb0b5-f15b-584e-cef7-7283a0e0729f

Article author

@SamMonoRT

@roji
Copy link
Member

roji commented Oct 7, 2024

Sure, I submitted #4829 to add a sentence to the SQLite limitations page. I'm skeptical this would actually help people catch a problem like this up-front, but it doesn't hurt.

@roji roji self-assigned this Oct 7, 2024
@roji roji added this to the 9.0.0 milestone Oct 7, 2024
@roji roji closed this as completed in #4829 Oct 8, 2024
@roji roji closed this as completed in cbe0130 Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants