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: Add ability for translation pipeline to translate property acc… #20427

Merged
merged 1 commit into from
Mar 28, 2020

Conversation

smitpatel
Copy link
Member

…ess over a subquery

This removes need to run subquery member pushdown after entity equality
Part of #20164
Part of #18923

This does not change anything on Cosmos as Cosmos does not have subquery scalar selection

@smitpatel smitpatel force-pushed the smit/IDidNotExpectItWouldComeToThis branch from 0959b51 to 555e4eb Compare March 27, 2020 15:22
@smitpatel
Copy link
Member Author

Updating cosmos also.

@smitpatel smitpatel force-pushed the smit/IDidNotExpectItWouldComeToThis branch from 555e4eb to 42d59e2 Compare March 27, 2020 20:37
@smitpatel
Copy link
Member Author

Updated.

@smitpatel smitpatel force-pushed the smit/IDidNotExpectItWouldComeToThis branch from 42d59e2 to d5522fc Compare March 27, 2020 22:41
@smitpatel smitpatel requested a review from maumar March 27, 2020 22:53
…ess over a subquery

This removes need to run subquery member pushdown after entity equality
Part of #20164
Part of #18923

Also improves Cosmos recursive binding over embedded navigations
@smitpatel smitpatel force-pushed the smit/IDidNotExpectItWouldComeToThis branch from d5522fc to 027bdb2 Compare March 28, 2020 06:03
@smitpatel smitpatel changed the base branch from smit/WhyYouDoThisToMe to master March 28, 2020 06:03
@smitpatel smitpatel merged commit 8771406 into master Mar 28, 2020
@smitpatel smitpatel deleted the smit/IDidNotExpectItWouldComeToThis branch March 28, 2020 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants