Skip to content

NH 3797 #432

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
merged 1 commit into from
Jun 23, 2015
Merged

NH 3797 #432

merged 1 commit into from
Jun 23, 2015

Conversation

PleasantD
Copy link
Contributor

See https://nhibernate.jira.com/browse/NH-3797

Modify the QueryModelVisitor to treat group by keys as first-class HQL candidates for select clause rewriting.

@hazzik hazzik modified the milestone: 4.1.0 Jun 17, 2015
@PleasantD PleasantD mentioned this pull request Jun 17, 2015
@PleasantD
Copy link
Contributor Author

Fixed the tabification. Please let me know how I can help expedite the next release.

…lause rewriting

- Added test cases
- Added the GroupByKeys expression list into the VisitorParameters class
- Created an extension method to simplify the extraction of group by key expressions
hazzik added a commit that referenced this pull request Jun 23, 2015
NH-3797 - Use group by keys as first class HqlCandidates for select clause rewriting
@hazzik hazzik merged commit f20a5e3 into nhibernate:master Jun 23, 2015
PleasantD added a commit to PleasantD/nhibernate-core that referenced this pull request Jul 7, 2015
…lauses

- Conditional statements are now generated in SQL for select clauses
- Case statements in the select clause don't generate unexpected or implicit joins for the test expression
- Combined the logic from SelectJoinDetector and ResultOperatorAndOrderByJoinDetector, also handles case statement join logic
- Removed most of the changes from NH-3797 (pull request nhibernate#432) as they are now unnecessary because the select clause now does the right thing
@PleasantD PleasantD mentioned this pull request Jul 7, 2015
@PleasantD PleasantD deleted the NH-3797 branch March 6, 2017 21:20
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.

2 participants