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: Translate List<T>.Count to server side #17735

Closed
smitpatel opened this issue Sep 10, 2019 · 0 comments · Fixed by #22897
Closed

Query: Translate List<T>.Count to server side #17735

smitpatel opened this issue Sep 10, 2019 · 0 comments · Fixed by #22897
Assignees
@maumar
Labels
area-query closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone
6.0.0

Comments

@smitpatel
Copy link
Member

We can convert it to Count() function. Currently it client evals materializing whole list.
@smitpatel smitpatel added this to the Backlog milestone Sep 27, 2019
@smitpatel smitpatel added the verify-fixed This issue is likely fixed in new query pipeline. label Mar 16, 2020
@bricelam bricelam modified the milestones: Backlog, MQ Sep 11, 2020
@maumar maumar added closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. and removed verify-fixed This issue is likely fixed in new query pipeline. labels Sep 24, 2020
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
4ec6d30 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
@maumar maumar mentioned this issue Oct 6, 2020
Merged
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
663a98a 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
dadbe42 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
89a46a9 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
bcf65fb 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
4be90a9 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
b17b781 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 6, 2020
@maumar
Additional regression tests for issues that have been fixed previously
8d1f793 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 7, 2020
@maumar
Additional regression tests for issues that have been fixed previously
03f661f 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
maumar added a commit that referenced this issue Oct 7, 2020
@maumar
Additional regression tests for issues that have been fixed previously
50e0118 
Resolves #10295
Resolves #12453
Resolves #13216
Resolves #13550
Resolves #13712
Resolves #13977
Resolves #15302
Resolves #17735
@maumar maumar closed this as completed in 2403579 Oct 7, 2020
@ajcvickers ajcvickers modified the milestones: MQ, 6.0.0 Nov 25, 2020
@ajcvickers ajcvickers modified the milestones: 6.0.0, 6.0.0-preview1 Jan 27, 2021
@ajcvickers ajcvickers removed this from the 6.0.0-preview1 milestone Nov 8, 2021
@ajcvickers ajcvickers added this to the 6.0.0 milestone Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maumar maumar

Labels
area-query 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
6.0.0
Development

Successfully merging a pull request may close this issue.

Additional regression tests for issues that have been fixed previously dotnet/efcore
4 participants
@bricelam @ajcvickers @smitpatel @maumar