Skip to content

NH-2184 - Projections / Arithmetic Operators #793

Open
@nhibernate-bot

Description

@nhibernate-bot

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.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions