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

Query: rewrite predicate with constanList.Any(e =>e == someProperty) to Contains #11389

Closed
maumar opened this issue Mar 22, 2018 · 2 comments
Closed
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. good first issue This issue should be relatively straightforward to fix. help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. type-enhancement
Milestone

Comments

@maumar
Copy link
Contributor

maumar commented Mar 22, 2018

We can fully translate contains, but Any will cause client eval currently

@smitpatel
Copy link
Member

Restrictions:

  • Has to be equality or inequality (includes operators & Equals method call both)
  • The constant list has to be of type which type mapper can generate literal for.
  • The lambda parameter needs to appear without any modification. (perhaps no convert etc)

@divega divega added this to the Backlog milestone Mar 23, 2018
@divega divega added the help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. label Mar 23, 2018
@smitpatel
Copy link
Member

merged via 98c8ac4

@smitpatel smitpatel modified the milestones: Backlog, 2.1.0 Mar 29, 2018
@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 29, 2018
@bricelam bricelam added the good first issue This issue should be relatively straightforward to fix. label May 31, 2019
@ajcvickers ajcvickers modified the milestones: 2.1.0-preview2, 2.1.0 Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. good first issue This issue should be relatively straightforward to fix. help wanted This issue involves technologies where we are not experts. Expert help would be appreciated. type-enhancement
Projects
None yet
Development

No branches or pull requests

5 participants