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.

Sign up for GitHub

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

Jump to bottom

Query: Add additional property to owned types in OwnedQueryTestBase #20343

Closed
smitpatel opened this issue Mar 19, 2020 · 1 comment · Fixed by #20353
Closed

Query: Add additional property to owned types in OwnedQueryTestBase #20343

smitpatel opened this issue Mar 19, 2020 · 1 comment · Fixed by #20353
Assignees
@smitpatel
Labels
area-query area-test closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone
5.0.0

Comments

@smitpatel
Copy link
Contributor

[6:42 PM] Andriy Svyryd
Can you convert that model to all owned types having 1 additional property mapped to a shared column?
smitpatel added a commit that referenced this issue Mar 19, 2020
@smitpatel
Query: Do not generate server side joins for optional dependents
bfc029a 
Resolves #18299

Fix involves:
- Adding additional materialization conditions for relational layer when table splitting is used. See #20332 for conditions
- Create EntityProjection using the same table when both types are sharing the table.
- Fix a bug in shaper when null entityType is returned from materialization condition, don't call StartTracking.
- Assign nullability to PK column correctly for optional dependents so we don't project it twice.
- Disabled Sql assertion in OwnedQuerySqlServerTest. Will enable after model update as described in #20336, #20343
smitpatel added a commit that referenced this issue Mar 19, 2020
@smitpatel
Query: Do not generate server side joins for optional dependents
5fe3628 
Fix involves:
- Adding additional materialization conditions for relational layer when table splitting is used. See #20332 for conditions
- Create EntityProjection using the same table when both types are sharing the table.
- Fix a bug in shaper when null entityType is returned from materialization condition, don't call StartTracking.
- Assign nullability to PK column correctly for optional dependents so we don't project it twice.
- Disabled Sql assertion in OwnedQuerySqlServerTest. Will enable after model update as described in #20336, #20343

Resolves #18299
Resolves #20338
Resolves #20332
@smitpatel smitpatel mentioned this issue Mar 19, 2020
Merged
@smitpatel smitpatel self-assigned this Mar 19, 2020
@smitpatel smitpatel added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Mar 19, 2020
@smitpatel smitpatel added this to the 5.0.0 milestone Mar 19, 2020
@smitpatel
Copy link
Contributor Author

Added a property which could be shared. Currently it is not sharing the column due to #20334
I will close this issue with added test data and add a note there. Also added issue number to commented code in tests.

@smitpatel smitpatel mentioned this issue Mar 19, 2020
Open
smitpatel added a commit that referenced this issue Mar 19, 2020
@smitpatel
Test: Update Sql assertion in OwnedQuerySqlServerTest
6f5cc15 
- Also add at least one additional property in each owned type. If there no columns other than shared PK then it is required dependent.
- Add a property in owned type which would be sharing column for hierarchy. Currently disabled due to #20334

Resolves #20343
Resolves #20336
Finishes left over test work from #20345
smitpatel added a commit that referenced this issue Mar 19, 2020
@smitpatel
Test: Update Sql assertion in OwnedQuerySqlServerTest
0008b80 
- Also add at least one additional property in each owned type. If there no columns other than shared PK then it is required dependent.
- Add a property in owned type which would be sharing column for hierarchy. Currently disabled due to #20334

Resolves #20343
Resolves #20336
Finishes left over test work from #20345
@smitpatel smitpatel mentioned this issue Mar 19, 2020
Merged
smitpatel added a commit that referenced this issue Mar 20, 2020
@smitpatel
Test: Update Sql assertion in OwnedQuerySqlServerTest
3d6861e 
- Also add at least one additional property in each owned type. If there no columns other than shared PK then it is required dependent.
- Add a property in owned type which would be sharing column for hierarchy. Currently disabled due to #20334

Resolves #20343
Resolves #20336
Finishes left over test work from #20345
@smitpatel smitpatel linked a pull request Mar 20, 2020 that will close this issue
Test: Update Sql assertion in OwnedQuerySqlServerTest #20353
Merged
smitpatel added a commit that referenced this issue Mar 20, 2020
@smitpatel
Query: Do not generate server side joins for optional dependents
0ccab1a 
Fix involves:
- Adding additional materialization conditions for relational layer when table splitting is used. See #20332 for conditions
- Create EntityProjection using the same table when both types are sharing the table.
- Fix a bug in shaper when null entityType is returned from materialization condition, don't call StartTracking.
- Assign nullability to PK column correctly for optional dependents so we don't project it twice.
- Disabled Sql assertion in OwnedQuerySqlServerTest. Will enable after model update as described in #20336, #20343

Resolves #18299
Resolves #20338
Resolves #20332
smitpatel added a commit that referenced this issue Mar 20, 2020
@smitpatel
Test: Update Sql assertion in OwnedQuerySqlServerTest
4ed67db 
- Also add at least one additional property in each owned type. If there no columns other than shared PK then it is required dependent.
- Add a property in owned type which would be sharing column for hierarchy. Currently disabled due to #20334

Resolves #20343
Resolves #20336
Finishes left over test work from #20345
smitpatel added a commit that referenced this issue Mar 20, 2020
@smitpatel
Query: Do not generate server side joins for optional dependents
a2e3531 
Fix involves:
- Adding additional materialization conditions for relational layer when table splitting is used. See #20332 for conditions
- Create EntityProjection using the same table when both types are sharing the table.
- Fix a bug in shaper when null entityType is returned from materialization condition, don't call StartTracking.
- Assign nullability to PK column correctly for optional dependents so we don't project it twice.
- Disabled Sql assertion in OwnedQuerySqlServerTest. Will enable after model update as described in #20336, #20343

Resolves #18299
Resolves #20338
Resolves #20332
@ajcvickers ajcvickers modified the milestones: 5.0.0, 5.0.0-preview3 Mar 31, 2020
@ajcvickers ajcvickers modified the milestones: 5.0.0-preview3, 5.0.0 Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smitpatel smitpatel

Labels
area-query area-test closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Projects
None yet
Milestone
5.0.0
Development

Successfully merging a pull request may close this issue.

Test: Update Sql assertion in OwnedQuerySqlServerTest dotnet/efcore
2 participants
@ajcvickers @smitpatel