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

[Transform] Add support for search-time runtime fields. #67643

Merged

Conversation

przemekwitek
Copy link
Contributor

@przemekwitek przemekwitek commented Jan 18, 2021

This PR adds support for specifying search-time runtime fields in SourceConfig so that these fields can be used by the pivot and latest transforms.

In this PR there is no validation against runtime fields being used as group_by, unique_key or sort but such a validation may be needed if we come to the conclusion that the performance degradation is significant.

Relates #65147

@przemekwitek przemekwitek force-pushed the transform_search_runtime_fields branch 4 times, most recently from 000f69e to 8973f3f Compare January 19, 2021 12:53
@przemekwitek przemekwitek removed the WIP label Jan 19, 2021
@przemekwitek przemekwitek marked this pull request as ready for review January 19, 2021 13:52
@przemekwitek przemekwitek force-pushed the transform_search_runtime_fields branch from 8973f3f to 6bd4e4c Compare January 19, 2021 13:56
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@przemekwitek przemekwitek force-pushed the transform_search_runtime_fields branch 2 times, most recently from 262474e to 88eb4bc Compare January 19, 2021 14:06
Copy link

@hendrikmuhs hendrikmuhs left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants