Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Abdon Pijpelink <[email protected]>
  • Loading branch information
szabosteve and abdonpijpelink authored Aug 23, 2023
1 parent 1237679 commit 0409194
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
3 changes: 2 additions & 1 deletion specification/ml/_types/Analysis.ts
Original file line number Diff line number Diff line change
Expand Up @@ -124,8 +124,9 @@ export class AnalysisConfigRead implements OverloadOf<AnalysisConfig> {
model_prune_window?: Duration
/**
* The size of the window in which to expect data that is out of time order.
* The default value is 0 (no latency).
* Defaults to no latency.
* If you specify a non-zero value, it must be greater than or equal to one second.
* @server_default 0
*/
latency?: Duration
/**
Expand Down
5 changes: 3 additions & 2 deletions specification/ml/_types/Job.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ export class Job {
daily_model_snapshot_retention_after_days?: long
/**
* The data description defines the format of the input data when you send data to the job by using the post data API.
* Note that when configure a datafeed, these properties are automatically set.
* Note that when configuring a datafeed, these properties are automatically set.
* When data is received via the post data API, it is not stored in Elasticsearch.
* Only the results for anomaly detection are retained.
*/
Expand Down Expand Up @@ -153,7 +153,8 @@ export class Job {
* Advanced configuration option, which affects the automatic removal of old model snapshots for this job.
* It specifies the maximum period of time (in days) that snapshots are retained.
* This period is relative to the timestamp of the most recent snapshot for this job.
* The default value is `10`, which means snapshots ten days older than the newest snapshot are deleted.
* By default, snapshots ten days older than the newest snapshot are deleted.
* @server_default 10
*/
model_snapshot_retention_days: long
/**
Expand Down

0 comments on commit 0409194

Please sign in to comment.