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
ESQL: Add CEIL
function
#98847
ESQL: Add CEIL
function
#98847
Conversation
The instructions for running them from Intellij may not work due to classpath issues etc. They do not work for devs using Intellij on Windows.
Documentation preview: |
* from within Intellij or, alternatively, via Gradle: | ||
* {@code ./gradlew -p x-pack/plugin/esql test --tests "org.elasticsearch.xpack.esql.CsvTests"} |
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.
Added this because, at least on my Windows, running from within Intellij IDEA does not work for some classpath/dependency resolution reasons.
@elasticsearchmachine run Check labels |
Pinging @elastic/es-ql (Team:QL) |
Hi @alex-spies, I've created a changelog YAML for you. |
Pinging @elastic/elasticsearch-esql (:Query Languages/ES|QL) |
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
Add generated signature and permitted types
@elasticsearchmachine run elasticsearch-ci/part-1 |
Hm, |
@elasticsearchmachine run elasticsearch-ci/part-1 |
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.
Left a couple of small docs suggestions, to put every command on its own line. Fixing it for FLOOR
too, while we're at it.
Add the unary scalar function
CEIL
.It takes a single double argument which is rounded up, and is a noop for integer arguments. Analogous to
FLOOR
.CEIL
, add it to the function registry and make sure it is serializable.CEIL
andFLOOR
FLOOR
.