-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
Avoid implicit casting in ESQL SearchStats #104947
Conversation
Pinging @elastic/es-analytical-engine (Team:Analytics) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - the result of this function is immediately cast back to long, so this avoids 2 unnecessary casts, in fact (one of which can be lossy).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@alex-spies @martijnvg Thanks for reviews. |
💚 Backport successful
|
./gradlew precommit fails with JDK21 and we should use longs instead of ints.
./gradlew precommit
fails with JDK21I think we should use longs instead of ints.