Skip to content

Conversation

@droberts195
Copy link

@droberts195 droberts195 commented Apr 28, 2020

This change adds a new setting, daily_model_snapshot_retention_after_days,
to the anomaly detection job config.

Initially this has no effect, the effect will be added in a followup PR.
This PR gets the complexities of making changes that interact with BWC
over well before feature freeze.

Relates #52150

This change adds a new setting, model_snapshot_retention_sparse_after_days,
to the anomaly detection job config.

Initially this has no effect, the effect will be added in a followup PR.
This PR gets the complexities of making changes that interact with BWC
over well before feature freeze.

Relates elastic#52150
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@benwtrent benwtrent self-requested a review April 28, 2020 17:04
@droberts195 droberts195 changed the title [ML] Add model_snapshot_retention_sparse_after_days to job config [ML] Add daily_model_snapshot_retention_after_days to job config Apr 28, 2020
@droberts195 droberts195 merged commit a74d822 into elastic:master Apr 28, 2020
@droberts195 droberts195 deleted the add_model_snapshot_retention_sparse_after_days branch April 28, 2020 19:56
droberts195 pushed a commit to droberts195/elasticsearch that referenced this pull request Apr 28, 2020
This change adds a new setting, daily_model_snapshot_retention_after_days,
to the anomaly detection job config.

Initially this has no effect, the effect will be added in a followup PR.
This PR gets the complexities of making changes that interact with BWC
over well before feature freeze.

Backport of elastic#55878
droberts195 pushed a commit that referenced this pull request Apr 29, 2020
)

This change adds a new setting, daily_model_snapshot_retention_after_days,
to the anomaly detection job config.

Initially this has no effect, the effect will be added in a followup PR.
This PR gets the complexities of making changes that interact with BWC
over well before feature freeze.

Backport of #55878
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