-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Whats new in EF9 docs #4823
Whats new in EF9 docs #4823
Conversation
a70c6ae
to
c41c259
Compare
_ = await GetPostsForceConstantCollection([1, 2, 3]); | ||
|
||
#region ForceConstantPrimitiveCollection | ||
async Task<List<Post>> GetPostsForceConstantCollection(int[] ids) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe better do something like:
var ids = [1, 2, 3];
var query = context.Posts
.Where(e => e.Title == ".NET Blog" && EF.Constant(ids).Contains(e.Id))
.ToListAsync();
rather than via method call? @roji
d70bff9
to
f73b7f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Mainly nits.
@@ -172,6 +172,8 @@ Math.Floor(d) | FLOOR(@d) | |||
Math.Log(d) | LOG(@d) | |||
Math.Log(a, newBase) | LOG(@a, @newBase) | |||
Math.Log10(d) | LOG10(@d) | |||
Math.Max(x, y) | GREATEST(@x, @y) | EF Core 9.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure any more, did we have an inferior translation here before or was it not working at all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not supported before:
The LINQ expression 'DbSet<NullSemanticsEntity>()
.Where(n => Math.Max(
val1: n.Int,
val2: 14) > 20)' could not be translated. Additional information: Translation of method 'System.Math.Max' failed.
entity-framework/core/what-is-new/ef-core-9.0/breaking-changes.md
Outdated
Show resolved
Hide resolved
entity-framework/core/what-is-new/ef-core-9.0/breaking-changes.md
Outdated
Show resolved
Hide resolved
entity-framework/core/what-is-new/ef-core-9.0/breaking-changes.md
Outdated
Show resolved
Hide resolved
entity-framework/core/what-is-new/ef-core-9.0/breaking-changes.md
Outdated
Show resolved
Hide resolved
FROM [Posts] AS [p] | ||
``` | ||
|
||
<a name="azuresql-azuresynapse"></a> | ||
|
||
### Better support for Azure SQL and Azure Synapse |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this one, I think we'll want to also document it inside the SQL Server provider docs (outside of the what's new) - and it would probably be a better idea to do both at the same time (and cross-link...), rather than having @cincuranet revisit this. But no big deal.
- Breaking change note for dotnet/efcore#33942 - Update function mappings Fixes #4765 Fixes #4805
also:
Nullable<T>.ToString()
efcore#33942Fixes #4765
Fixes #4805