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

Enable required dependents when using table splitting #15607

Closed
ajcvickers opened this issue May 3, 2019 · 2 comments
Closed

Enable required dependents when using table splitting #15607

ajcvickers opened this issue May 3, 2019 · 2 comments

Comments

@ajcvickers
Copy link
Member

See #9005 and #15521

Note that this does not mean requireness needs to be enforced. It just means that we assume if the relationship is configured as required that we don't need to do anything special to enable it to be optional.

@ajcvickers
Copy link
Member Author

ajcvickers commented Jul 11, 2019

Talked again to @AndriySvyryd and @smitpatel on this. Despite what I said in May I now understand why it makes sense to punt this.

This change would allow a dependent in a split table to have all null values for it's properties, and yet still get an entity instance materialized. This is not the same as making the relationship required, since that still allows a principal to exist without a dependent. This therefore requires new API, and is, relatively speaking a corner case.

This could be accomplished with an API that does not allow the principle exist without the related dependent also existing. This in turn would allow required properties in the dependent to map to non-nullable columns, since every row must contain properties for both entities, always.

@AndriySvyryd
Copy link
Member

Duplicate of #12100

@AndriySvyryd AndriySvyryd marked this as a duplicate of #12100 Sep 8, 2019
@AndriySvyryd AndriySvyryd removed this from the Backlog milestone Sep 8, 2019
@divega divega closed this as completed Sep 9, 2019
@AndriySvyryd AndriySvyryd removed their assignment Nov 27, 2019
@ajcvickers ajcvickers reopened this Oct 16, 2022
@ajcvickers ajcvickers closed this as not planned Won't fix, can't repro, duplicate, stale Oct 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants