Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport c30d5d0 from #4138.

* Support refering to implicit @timestamp field in time-based aggregations

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Update documentation of stats to reflect that span can be used without specifying a field

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Move @timestamp reference to AST layer

- Additionally refactored visitTimechartCommand to reuse spanLiteral definition

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Unit test visitSpanLiteral, vistSpanClause, and visitTimechartParamter

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Revert changes to Span will always have a field with the current implementation

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

* Throw exception for zero span

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

---------

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
(cherry picked from commit c30d5d0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
@qianheng-aws qianheng-aws merged commit 2c41580 into 2.19-dev Oct 21, 2025
71 of 72 checks passed
@yuancu yuancu deleted the backport/backport-4138-to-2.19-dev branch October 21, 2025 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants