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

[ML] Support rolled-up indices in datafeeds #31999

Closed
dimitris-athanasiou opened this issue Jul 12, 2018 · 4 comments
Closed

[ML] Support rolled-up indices in datafeeds #31999

dimitris-athanasiou opened this issue Jul 12, 2018 · 4 comments
Assignees
Labels
>feature :ml Machine learning

Comments

@dimitris-athanasiou
Copy link
Contributor

As users start using roll-ups to downsize their indices, it makes sense for datafeeds to be able to fetch data from rolled-up indices.

@dimitris-athanasiou dimitris-athanasiou added >feature :ml Machine learning labels Jul 12, 2018
@dimitris-athanasiou dimitris-athanasiou self-assigned this Jul 12, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@dimitris-athanasiou
Copy link
Contributor Author

A quick look at that revealed that we cannot scroll using rollup-search. I think this means we can only support rolled-up indices when using aggregations.

Also, it is worth noting that instead of using field-caps, for rollups we should make use of the rollup-caps API to check if the analyzed fields are available.

Another question is how we handle index patterns that match both raw and rollup indices.

@droberts195
Copy link
Contributor

Notes from our discussion about how to implement this:

  • No datafeed config changes are required
  • We will use rollup search if the rollup caps action finds any rollup indices in the indices list in the datafeed config
  • We will throw an error if rollup caps finds any rolled up indices and the datafeed config does not contain aggregations or if the datafeed date histogram interval is not a multiple of the rollup date histogram interval
  • We will throw an error if rollup caps finds any rolled up indices and they cannot service the datafeed's aggregation
  • We will create an auditor notification when it is decided that a datafeed will use rollups
  • Make sure the preview endpoint works for rollups

@benwtrent
Copy link
Member

This issue was closed with PR: #34654

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>feature :ml Machine learning
Projects
None yet
Development

No branches or pull requests

4 participants