Skip to content

[Derived Fields] Add aggregation support for derived fields #14618

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Jul 29, 2024
Merged

[Derived Fields] Add aggregation support for derived fields #14618

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
c437517
Add aggregation support for derived fields
mch2 Jul 1, 2024
27c0389
add unit test for a terms agg with derived fields
mch2 Jul 22, 2024
d4d958a
Fix license header and add changelog entry
mch2 Jul 22, 2024
9374f86
move matrix_stats tests to aggs-matrix-stats module
mch2 Jul 23, 2024
af28c9d
Move matrix tests back and add dependency to painless module
mch2 Jul 23, 2024
e64b6f5
add tests for all aggregations types and support ip_range
mch2 Jul 23, 2024
30d4240
Add tests for agg script returned from DerivedFieldType
mch2 Jul 24, 2024
632e88c
remove children aggs test as its not yet supported
mch2 Jul 24, 2024
f0babf4
Add more tests
mch2 Jul 24, 2024
e4304eb
fix changelog
mch2 Jul 24, 2024
f71f55d
Merge branch 'main' into derivedaggs
mch2 Jul 24, 2024
7fcab64
Merge branch 'main' into derivedaggs
mch2 Jul 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
## [Unreleased 2.x]
### Added
- Fix for hasInitiatedFetching to fix allocation explain and manual reroute APIs (([#14972](https://github.com/opensearch-project/OpenSearch/pull/14972))
- Add basic aggregation support for derived fields ([#14618](https://github.com/opensearch-project/OpenSearch/pull/14618))

### Dependencies
- Bump `org.apache.commons:commons-lang3` from 3.14.0 to 3.15.0 ([#14861](https://github.com/opensearch-project/OpenSearch/pull/14861))
Expand Down
1 change: 1 addition & 0 deletions modules/lang-painless/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ ext {

testClusters.all {
module ':modules:mapper-extras'
module ':modules:aggs-matrix-stats'
systemProperty 'opensearch.scripting.update.ctx_in_params', 'false'
// TODO: remove this once cname is prepended to transport.publish_address by default in 8.0
systemProperty 'opensearch.transport.cname_in_publish_address', 'true'
Expand Down
Loading