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

Split query generation/docs [possible repeat] #4046

Closed
DickBaker opened this issue Sep 20, 2022 · 1 comment
Closed

Split query generation/docs [possible repeat] #4046

DickBaker opened this issue Sep 20, 2022 · 1 comment

Comments

@DickBaker
Copy link
Contributor

  1. the code examples will bring back all the Blogs & Posts rows. Better that examples have a WHERE Rating>=4 or somesuch to show how that will also be present in the 2nd SELECT split.
  2. the "cartesian explosion" duplicates only applies when querying Posts with an INCLUDE("Blogs") that will have dups of Blogs with multiple Posts, so I would suggest you revise the examples given.
  3. surely the EF code generation should impose a DISTINCT in the split statements to suppress such dups?
  4. I am surprised that the generated split code shown uses the b.BlogId and not the p.BlogId - yes I know they are the same per the ON clause but it rankles when you are really looking for all the p.* fields to match the EF entity definition [just requires more brain-cycles to validate the resulant match!]

Apologies if this shows up as a repeat as I can't see Issue against my Participating filter. Maybe I never hit the Submit button [brain fade] or EF team summarily Closed it?!
[Enter feedback here]


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

@roji
Copy link
Member

roji commented Sep 20, 2022

Duplicate of #4044

@roji roji marked this as a duplicate of #4044 Sep 20, 2022
@roji roji closed this as not planned Won't fix, can't repro, duplicate, stale Sep 20, 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

2 participants