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

InMemory: Use single query to server #17106

Closed
smitpatel opened this issue Aug 12, 2019 · 1 comment
Closed

InMemory: Use single query to server #17106

smitpatel opened this issue Aug 12, 2019 · 1 comment
Assignees
@smitpatel
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-cleanup
Milestone
3.0.0

Comments

@smitpatel
Copy link
Contributor

Currently our server query expression contains another QueryingEnumerable inside it. Essentially making it N+1 queries. No functional impact. It would be good to have stricter boundaries between what is server side and what is client side for InMemory like other providers.
@smitpatel smitpatel mentioned this issue Aug 12, 2019
Merged
@ajcvickers ajcvickers added this to the Backlog milestone Aug 14, 2019
@smitpatel
Copy link
Contributor Author

Fixed in #17366

@smitpatel smitpatel removed this from the Backlog milestone Aug 23, 2019
@smitpatel smitpatel self-assigned this Aug 23, 2019
@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 Aug 23, 2019
@smitpatel smitpatel added this to the 3.0.0 milestone Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smitpatel smitpatel

Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-cleanup
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
2 participants
@ajcvickers @smitpatel