Skip to content

Conversation

@pabloem
Copy link
Contributor

@pabloem pabloem commented Sep 10, 2025

I implemented the idelta function by adding a parameter for the aggregate function in irate that determines whether or not to divide by time.

The idea will be something similar for delta.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.2.0 labels Sep 10, 2025
@pabloem pabloem added >feature :StorageEngine/TSDB You know, for Metrics Team:StorageEngine and removed needs:triage Requires assignment of a team area label labels Sep 10, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@pabloem pabloem requested a review from dnhatn September 11, 2025 16:38
@pabloem pabloem requested a review from kkrik-es September 11, 2025 22:30
Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Can you add csv tests? Thanks!

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great.

@pabloem pabloem merged commit da70398 into elastic:main Sep 12, 2025
34 checks passed
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Sep 17, 2025
* Implementing idelta function for esql

* fixes for idelta

* fixes for idelta

* [CI] Auto commit changes from spotless

* csv tests

* comments

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
gmjehovich pushed a commit to gmjehovich/elasticsearch that referenced this pull request Sep 18, 2025
* Implementing idelta function for esql

* fixes for idelta

* fixes for idelta

* [CI] Auto commit changes from spotless

* csv tests

* comments

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants