-
Notifications
You must be signed in to change notification settings - Fork 517
Query: Adds translation support for single key single value select GROUP BY LINQ queries #4074
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
Merged
microsoft-github-policy-service
merged 39 commits into
master
from
users/leminh/LINQGroupBySupport
Apr 1, 2024
Merged
Changes from 5 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
e1ceebb
preliminary change
8c2d289
Add some more boiler plate code
ccba9f2
move all linq test to the same folder; add some groupBy test
72b315c
fix references error in test refactoring
d962b02
preliminary for the groupby functions with key and value selector
36d668e
trying to change collection inputs for group by
73ea3c6
WIP bookmark
dc50d78
Successfully ignore "key"
e42ae59
clean up code
2244764
Sucessfully bind the case of group by with only key selector and no v…
df9362e
enable one group by test
7790ab7
merge with master
da48bfc
Merge branch 'master' into users/leminh/LINQGroupBySupport
44d3736
add support for aggregate value selector
7631e82
added baseline
b4b1658
working on adding support for multivalue value selector and key selector
787839d
code clean up
7da5772
Merge branch 'master' into users/leminh/LINQGroupBySupport
e8f8f62
more clean up
f589069
more clean up
bca5953
update test
544931a
Move test to separate file
a7888d8
code clean up
6b7fb50
remove baseline file that got moved
d23c29d
merge with master
5a4470b
fix merge issue
84e4c2f
merge with master
1e80e2d
Changes test infrastructure to reflect changes from Master
0b08efd
address code review part 1
9a6eacf
Address code review 2 and adds code coverage
f73baf1
Merge branch 'master' into users/leminh/LINQGroupBySupport
leminh98 9f77356
Addressed code review and added tests. Still a couple of bugs to iron…
105a335
Merge branch 'users/leminh/LINQGroupBySupport' of https://github.com/…
25c7080
resolve merge conflict with master
430f96a
Fix group by translation issue and add more test
d1ec2c5
update comments
fc32e3a
address pr comment
33a2c9b
Merge branch 'master' into users/leminh/LINQGroupBySupport
leminh98 2444be7
Merge branch 'master' into users/leminh/LINQGroupBySupport
adityasa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.