Skip to content

Conversation

@roji
Copy link
Member

@roji roji commented Aug 22, 2025

Part of #36296

Description

This fixes queries which involve structural containment for complex JSON collections:

var post = ...;
var blogs = context.Blogs.Where(b => b.Posts.Contains(post)).ToListAsync();

This sort of query currently fails when b.Posts is mapped as complex JSON. Making this work properly is part of the complex type/JSON push in 10.

Customer impact

All queries of the above form currently fail.

How found

Testing.

Regression

No

Testing

Tests added

Risk

Low, the fix almost exclusively touches new complex type/JSON code that was added in 10.

@roji roji requested a review from artl93 August 22, 2025 11:01
@roji roji requested a review from a team as a code owner August 22, 2025 11:01
@roji roji added the ask-mode label Aug 22, 2025
@roji roji force-pushed the MoreComplexTypeScenarios branch from 4bad64a to 505e657 Compare August 22, 2025 14:15
@roji roji enabled auto-merge (squash) August 22, 2025 15:04
@roji roji disabled auto-merge August 22, 2025 15:04
Copy link
Member

@artl93 artl93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rc2

@roji roji merged commit d3f3e00 into dotnet:release/10.0 Aug 22, 2025
7 checks passed
@roji roji deleted the MoreComplexTypeScenarios branch August 22, 2025 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants