Skip to content
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-DataFrame] add sync api #41800

Conversation

hendrikmuhs
Copy link

Feature Branch PR

Introduces the synchronization API and adds a implementation for time-based synchronization.

Time-based synchronization means we require the existence of a time field while new data arrives with this field set accordingly with no updates, no deletes.

The API with an example looks as follows:

"sync" : {
  "time" : {
   "field" : "@timestamp",
   "delay": "5m"
  }
}

Other synchronization methods - instead of time - can be plugged in later, that's why fields below sync are implemented as named writables.

This PR adds the API without an implementation.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@hendrikmuhs hendrikmuhs changed the title [ML-DataFrame] Feature ml data frame analytics sync api [ML-DataFrame] add sync api May 3, 2019
@hendrikmuhs
Copy link
Author

run elasticsearch-ci/1

@benwtrent benwtrent self-requested a review May 6, 2019 11:59
@hendrikmuhs hendrikmuhs merged commit 362775b into elastic:feature-ml-data-frame-analytics May 6, 2019
hendrikmuhs pushed a commit that referenced this pull request Jul 16, 2019
PR 41491 hasn't been merged into 7.2 but got closed and superseeded by #41800 (merged via feature branch merge into 7.3)
jrodewig pushed a commit that referenced this pull request Jul 16, 2019
PR 41491 hasn't been merged into 7.2 but got closed and superseeded by #41800 (merged via feature branch merge into 7.3)
lcawl pushed a commit that referenced this pull request Jul 16, 2019
PR 41491 hasn't been merged into 7.2 but got closed and superseeded by #41800 (merged via feature branch merge into 7.3)
lcawl pushed a commit that referenced this pull request Jul 16, 2019
PR 41491 hasn't been merged into 7.2 but got closed and superseeded by #41800 (merged via feature branch merge into 7.3)
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.

3 participants