-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Set data recognizer to ignore frozen indices #117208
Conversation
Pinging @elastic/ml-ui (:ml) |
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: cc @qn895 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Co-authored-by: Quynh Nguyen <[email protected]>
It's just been pointed out to me that the default for
I suspect that "frozen" in the context that caused the problem to be observed meant frozen tier aka searchable snapshots, and not frozen indices (which is what
If the problem really was with frozen tier aka searchable snapshots then the solution is instead to change the data recognizer to only search data from the most recent 30 or 90 days. Ideally this would be the most recent N days defined by the ILM policy that moves data into the frozen tier after N days. However, until there is an API that can answer the question of what that N is we'll have to go with a hardcoded 30 or 90 based on common practice for when data is moved to frozen. |
Today I learnt about elastic/elasticsearch#69288. Frozen tier (aka searchable snapshots) is the modern way to manage old data and is completely different to frozen indices which is old functionality that's being deprecated in 8.0. I think instead of setting |
This reverts commit 2f24d14
This reverts commit 2f24d14
This reverts commit 2f24d14 Co-authored-by: Quynh Nguyen <[email protected]>
Summary
Addresses #116696. Currently, it will take a long time to get results from /api/ml/modules/recognize especially if the index pattern includes indices that are frozen. This PR sets
ignore_throttled
to true as we would not need to search for frozen. This should speed up the request.