Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Sep 26, 2018

Related to #33961 and #33565

This PR adds documentation for the new dynamic settings.

@lcawl lcawl added >docs General docs changes WIP v7.0.0 :ml Machine learning v6.5.0 labels Sep 26, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@lcawl lcawl removed the WIP label Sep 27, 2018
@lcawl
Copy link
Contributor Author

lcawl commented Sep 27, 2018

@droberts195 and @benwtrent: One thing that wasn't clear to me was whether the percentage of date/order errors is relative to a subset of the data (e.g. per bucket). If we can clarify that in the description, that'd be great.

The maximum number of records that are output per bucket. The default value is
`500`.

`xpack.ml.max_percent_date_errors`:: (<<cluster-update-settings,Dynamic>>)

Choose a reason for hiding this comment

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

It turns out that this setting and xpack.ml.max_percent_out_of_order_errors are never used anywhere. (They used to be used in the old Prelert product.)

So they should be removed completely from the docs and code.

@lcawl
Copy link
Contributor Author

lcawl commented Sep 28, 2018

Thanks @droberts195, I've removed those two settings.

Copy link

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

@lcawl lcawl merged commit f143e39 into elastic:master Sep 28, 2018
@lcawl lcawl deleted the lcawley-dynamic-settings branch September 28, 2018 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes :ml Machine learning v6.5.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants