Skip to content

[ML] Add integration tests to check persistence #40272

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

Merged
merged 2 commits into from
Mar 21, 2019

Conversation

edsavage
Copy link
Contributor

Additional checks have been added to exercise the behaviour of
persistence on graceful close of an anomaly job. In particular:

  • check that persistence does not occur for a job that is opened and
    then immediately closed, with nothing else having happened.
  • check that persistence occurs on graceful close of a job if it has
    processed data.
  • check that persistence occurs subsequent to time being manually
    advanced - even if no additional data has been seen by the job
  • check the edge case where persistence occurs if a job is opened, time
    is manually advanced and then the job is closed, having seen no data.

Related to elastic/ml-cpp#393

Additional checks have been added to exercise the behaviour of
persistence on graceful close of an anomaly job. In particular:

 - check that persistence does not occur for a job that is opened and
 then immediately closed, with nothing else having happened.
 - check that persistence occurs on graceful close of a job if it has
 processed data.
 - check that persistence occurs subsequent to time being manually
 advanced - even if no additional data has been seen by the job
 - check the edge case where persistence occurs if a job is opened, time
 is manually advanced and then the job is closed, having seen no data.

 Related to elastic/ml-cpp#393
@edsavage edsavage added >test Issues or PRs that are addressing/adding tests v7.0.0 :ml Machine learning v6.7.0 v8.0.0 v7.2.0 labels Mar 20, 2019
@edsavage edsavage requested a review from droberts195 March 20, 2019 17:27
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

Copy link
Contributor

@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

@edsavage
Copy link
Contributor Author

Jenkins retest this please

@edsavage
Copy link
Contributor Author

Jenkins retest this please

@edsavage edsavage merged commit b0ce522 into elastic:master Mar 21, 2019
edsavage added a commit to edsavage/elasticsearch that referenced this pull request Mar 21, 2019
Additional checks have been added to exercise the behaviour of
persistence on graceful close of an anomaly job. In particular:

 - check that persistence does not occur for a job that is opened and
 then immediately closed, with nothing else having happened.
 - check that persistence occurs on graceful close of a job if it has
 processed data.
 - check that persistence occurs subsequent to time being manually
 advanced - even if no additional data has been seen by the job
 - check the edge case where persistence occurs if a job is opened, time
 is manually advanced and then the job is closed, having seen no data.

 Related to elastic/ml-cpp#393
edsavage added a commit to edsavage/elasticsearch that referenced this pull request Mar 21, 2019
Additional checks have been added to exercise the behaviour of
persistence on graceful close of an anomaly job. In particular:

 - check that persistence does not occur for a job that is opened and
 then immediately closed, with nothing else having happened.
 - check that persistence occurs on graceful close of a job if it has
 processed data.
 - check that persistence occurs subsequent to time being manually
 advanced - even if no additional data has been seen by the job
 - check the edge case where persistence occurs if a job is opened, time
 is manually advanced and then the job is closed, having seen no data.

 Related to elastic/ml-cpp#393
edsavage added a commit that referenced this pull request Mar 21, 2019
Additional checks to exercise the behaviour of
persistence on graceful close of an anomaly job.

Related to elastic/ml-cpp#393
Backports #40272
edsavage added a commit that referenced this pull request Mar 21, 2019
Additional checks to exercise the behaviour of
persistence on graceful close of an anomaly job

Related to elastic/ml-cpp#393
Backports #40272
edsavage added a commit that referenced this pull request Mar 21, 2019
Additional checks to exercise the behaviour of
persistence on graceful close of an anomaly job.

Related to elastic/ml-cpp#393
Backports #40272
@edsavage edsavage deleted the persist_state_on_graceful_job_close branch April 12, 2019 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning >test Issues or PRs that are addressing/adding tests v6.7.1 v7.0.0-rc1 v7.2.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants