Skip to content

[Star Tree] [Search] Resolving Range aggregations with Star-tree#17273

Merged
msfroh merged 4 commits intoopensearch-project:mainfrom
sandeshkr419:r1
Apr 8, 2025
Merged

[Star Tree] [Search] Resolving Range aggregations with Star-tree#17273
msfroh merged 4 commits intoopensearch-project:mainfrom
sandeshkr419:r1

Conversation

@sandeshkr419
Copy link
Copy Markdown
Member

@sandeshkr419 sandeshkr419 commented Feb 6, 2025

Description

Resolving numeric range aggregation queries via star-tree.
Supporting numeric range aggregations with/without metric sub-aggregations.

Link to comment highlighting benchmarking comparisons: #17273 (comment)

Temp Notes

This PR contains unmerged changes keywords aggregation PR from as well. Kindly only look into range aggs last commit changes commit while reviewing the changes for now.

(Depending changes merged)

Related Issues

Resolves #16553

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions github-actions bot added enhancement Enhancement or improvement to existing feature or request Search:Aggregations labels Feb 6, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 6, 2025

❌ Gradle check result for 4e15156: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2025

❌ Gradle check result for 0d58fe5: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@sandeshkr419 sandeshkr419 reopened this Apr 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2025

❌ Gradle check result for 0d58fe5: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@sandeshkr419 sandeshkr419 reopened this Apr 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2025

✅ Gradle check result for 0d58fe5: SUCCESS

@msfroh msfroh merged commit 4560206 into opensearch-project:main Apr 8, 2025
83 of 87 checks passed
@sandeshkr419 sandeshkr419 deleted the r1 branch April 8, 2025 03:40
guojialiang92 pushed a commit to guojialiang92/OpenSearch that referenced this pull request Apr 8, 2025
…nsearch-project#17273)

* range aggs changes

---------

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
pranu2502 pushed a commit to pranu2502/OpenSearch that referenced this pull request Apr 8, 2025
…nsearch-project#17273)

* range aggs changes

---------

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
Harsh-87 pushed a commit to Harsh-87/OpenSearch that referenced this pull request May 7, 2025
…nsearch-project#17273)

* range aggs changes

---------

Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
Signed-off-by: Harsh Kothari <techarsh@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement or improvement to existing feature or request Search:Aggregations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Star Tree] [Search] Range Aggregations with/without metric sub-aggregation

3 participants