Skip to content
Merged
Changes from 3 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
5370876
Merge pull request #36540 from dotnet/release/8.0
cincuranet Aug 8, 2025
e533e1f
Update dependencies from https://github.com/dotnet/arcade build 20250…
dotnet-maestro[bot] Aug 18, 2025
fa88a70
Update dependencies from https://github.com/dotnet/arcade build 20250…
dotnet-maestro[bot] Sep 1, 2025
103993f
Update branding to 9.0.10 (#36697)
vseanreesermsft Sep 2, 2025
a5577ac
Update branding to 8.0.21 (#36696)
vseanreesermsft Sep 2, 2025
467e6e7
Merge branch 'release/9.0' into merge/release/8.0-to-release/9.0
cincuranet Sep 2, 2025
0502505
Merge pull request #36699 from dotnet/merge/release/8.0-to-release/9.0
cincuranet Sep 2, 2025
2eb3bba
Merging internal commits for release/8.0 (#36727)
vseanreesermsft Sep 10, 2025
003be39
Merging internal commits for release/9.0 (#36728)
vseanreesermsft Sep 10, 2025
4b15ce0
Merge branch 'release/9.0' into merge/release/8.0-to-release/9.0
cincuranet Sep 10, 2025
902c605
Merge pull request #36734 from dotnet/merge/release/8.0-to-release/9.0
cincuranet Sep 10, 2025
bd6018a
Update dependencies from https://github.com/dotnet/arcade build 20250…
dotnet-maestro[bot] Sep 11, 2025
e639f77
Update dependencies from https://github.com/dotnet/arcade build 20250…
dotnet-maestro[bot] Sep 12, 2025
7a36174
Update dependencies from https://github.com/dotnet/arcade build 20250…
dotnet-maestro[bot] Sep 15, 2025
8378c07
[release/8.0] Disable Guardian Dotnet Analyzers (#36776)
AndriySvyryd Sep 15, 2025
25099bf
[release/9.0] Disable Guardian Dotnet Analyzers (#36777)
AndriySvyryd Sep 15, 2025
d574040
Resolve query logger from QueryContext in shaper generation (#36758) …
roji Sep 15, 2025
63815d3
Merge pull request #36788 from dotnet/release/8.0-staging
cincuranet Sep 15, 2025
de3da92
Merge pull request #36789 from dotnet/release/9.0-staging
cincuranet Sep 15, 2025
2319935
Merge branch 'release/9.0' into merge/release/8.0-to-release/9.0
cincuranet Sep 16, 2025
b401088
Merge pull request #36784 from dotnet/merge/release/8.0-to-release/9.0
cincuranet Sep 16, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ private sealed partial class ShaperProcessingExpressionVisitor : ExpressionVisit
public static readonly bool UseOldBehavior32310 =
AppContext.TryGetSwitch("Microsoft.EntityFrameworkCore.Issue32310", out var enabled32310) && enabled32310;

private static readonly bool UseOldBehavior36464 =
AppContext.TryGetSwitch("Microsoft.EntityFrameworkCore.Issue36464", out var enabled36464) && enabled36464;

/// <summary>
/// Reading database values
/// </summary>
Expand Down Expand Up @@ -80,6 +83,9 @@ private static readonly MethodInfo Utf8JsonReaderGetStringMethod
private static readonly MethodInfo EnumParseMethodInfo
= typeof(Enum).GetMethod(nameof(Enum.Parse), new[] { typeof(Type), typeof(string) })!;

private static readonly PropertyInfo QueryContextQueryLoggerProperty =
typeof(QueryContext).GetProperty(nameof(QueryContext.QueryLogger))!;

private readonly RelationalShapedQueryCompilingExpressionVisitor _parentVisitor;
private readonly ISet<string>? _tags;
private readonly bool _isTracking;
Expand Down Expand Up @@ -1644,7 +1650,9 @@ protected override Expression VisitSwitch(SwitchExpression switchExpression)
New(
JsonReaderManagerConstructor,
_jsonReaderDataParameter,
Constant(_queryLogger))),
UseOldBehavior36464
? Constant(_queryLogger)
: MakeMemberAccess(QueryCompilationContext.QueryContextParameter, QueryContextQueryLoggerProperty))),
// tokenType = jsonReaderManager.CurrentReader.TokenType
Assign(
tokenTypeVariable,
Expand Down Expand Up @@ -1807,7 +1815,13 @@ void ProcessFixup(IDictionary<string, LambdaExpression> fixupMap)
var captureState = Call(managerVariable, Utf8JsonReaderManagerCaptureStateMethod);
var assignment = Assign(propertyVariable, innerShaperMapElement.Value);
var managerRecreation = Assign(
managerVariable, New(JsonReaderManagerConstructor, _jsonReaderDataParameter, Constant(_queryLogger)));
managerVariable,
New(
JsonReaderManagerConstructor,
_jsonReaderDataParameter,
UseOldBehavior36464
? Constant(_queryLogger)
: MakeMemberAccess(QueryCompilationContext.QueryContextParameter, QueryContextQueryLoggerProperty)));

readExpressions.Add(
Block(
Expand Down Expand Up @@ -2170,7 +2184,13 @@ private static IList<T> PopulateList<T>(IList<T> buffer, IList<T> target)
Default(typeof(JsonReaderData))),
Block(
Assign(
jsonReaderManagerVariable, New(JsonReaderManagerConstructor, jsonReaderDataVariable, Constant(_queryLogger))),
jsonReaderManagerVariable,
New(
JsonReaderManagerConstructor,
jsonReaderDataVariable,
UseOldBehavior36464
? Constant(_queryLogger)
: MakeMemberAccess(QueryCompilationContext.QueryContextParameter, QueryContextQueryLoggerProperty))),
Call(jsonReaderManagerVariable, Utf8JsonReaderManagerMoveNextMethod),
Call(jsonReaderManagerVariable, Utf8JsonReaderManagerCaptureStateMethod)));

Expand Down
Loading