Skip to content

NH-2184 - Projections / Arithmetic Operators #793

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

Open
nhibernate-bot opened this issue Oct 12, 2017 · 0 comments
Open

NH-2184 - Projections / Arithmetic Operators #793

nhibernate-bot opened this issue Oct 12, 2017 · 0 comments

Comments

@nhibernate-bot
Copy link
Collaborator

nhibernate-bot commented Oct 12, 2017

Mihai Codrean created an issue — 22nd April 2010, 6:09:48:

Implemented the binary operator projections: addition, subtraction, multiplication and division.
Basic test included.

Change provided as a patch on JIRA.

patch.txt


Onur Gumus added a comment — 3rd July 2011, 8:55:34:

This looks pretty cool. Would you please commit this ?


Mihai Codrean added a comment — 4th July 2011, 1:51:52:

Thanks Onur! However, I am not a committer, so we have to wait for one to do this.

http://en.wikipedia.org/wiki/Committer


Julian Maughan added a comment — 7th August 2011, 5:54:35:

I've had a look at this patch (a little while ago now actually) and don't think it is ready for commit. Particularly it does not address what the correct return type of the projection should be. For example, the patch makes it possible to perform a binary operation (plus, minus, multiply or divide) on the values of two column projections, which could project columns of a different type. If one column is a decimal and the other an int, the code needs to derive an appropriate return type - in this case, a decimal to avoid any loss of precision.

Also from memory, I don't think GetTypes was implemented correctly: returning multiple types when a binary operator should only have one return type.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants