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

Long priority over Float #18282

Merged
merged 2 commits into from May 12, 2016
Merged

Long priority over Float #18282

merged 2 commits into from May 12, 2016

Conversation

jdconrad
Copy link
Contributor

Def math operations will now check for Long before Float. Note that type promotion still works correctly because we make sure both ops are Long. (This does require an extra branch in some cases, but I believe the trade off is worth it as Float should be a very rarely used type.)

@rjernst
Copy link
Member

rjernst commented May 11, 2016

LGTM

@rmuir
Copy link
Contributor

rmuir commented May 11, 2016

maybe we can be a little more conservative and reorder only where it won't cost us an additional branch?

I think its helpful to keep the cases minimal too in case we implement them differently (e.g. methodhandle).

@rmuir
Copy link
Contributor

rmuir commented May 11, 2016

+1

@jdconrad jdconrad merged commit 8871fdd into elastic:master May 12, 2016
@jdconrad jdconrad deleted the longs branch May 12, 2016 07:57
@clintongormley clintongormley added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache and removed :Plugin Lang Painless labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >enhancement v5.0.0-alpha3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants