Skip to content
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

[Derived Fields] Add Aggregation support for Derived Fields #14590

Closed
Tracked by #12281
mch2 opened this issue Jun 27, 2024 · 0 comments · Fixed by #14618
Closed
Tracked by #12281

[Derived Fields] Add Aggregation support for Derived Fields #14590

mch2 opened this issue Jun 27, 2024 · 0 comments · Fixed by #14618

Comments

@mch2
Copy link
Member

mch2 commented Jun 27, 2024

Is your feature request related to a problem? Please describe
Now that we have basic derived fields we can add aggregation support for them.

Describe the solution you'd like
Ideally all existing aggregatios will extend to support DerivedField and DerivedField object types.

I have started working on getting this rolling with basic bucket and metric aggs by adding a method to fetch an AggregationScript.LeafFactory directly from DerivedFieldType. This will return a script to be used with valueSources that fetches values from DerivedFieldValueFetcher.

Related component
Search

Describe alternatives you've considered
Ignore agg support for derived fields.

@mch2 mch2 mentioned this issue Jun 27, 2024
6 tasks
@mch2 mch2 changed the title Aggregation support [Feature Request] Add Aggregation support for Derived Fields Jun 27, 2024
@mch2 mch2 changed the title [Feature Request] Add Aggregation support for Derived Fields [Derived Fields] Add Aggregation support for Derived Fields Jun 27, 2024
@mch2 mch2 self-assigned this Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants