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.

Sign up for GitHub

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

Jump to bottom

[6.5.3][ML] Correct query times for model plot and forecast #329

Merged
merged 1 commit into from
Dec 5, 2018
Merged

[6.5.3][ML] Correct query times for model plot and forecast #329

merged 1 commit into from
Dec 5, 2018

Conversation

tveasey
Copy link
Contributor

@tveasey tveasey commented Dec 5, 2018 •
edited
Loading

Backport #327.

@tveasey tveasey merged commit c53a6f0 into elastic:6.5 Dec 5, 2018
@tveasey tveasey changed the title [ML] Correct query times for model plot and forecast (#327) [ML] Correct query times for model plot and forecast Dec 5, 2018
droberts195 added a commit that referenced this pull request Dec 6, 2018
@droberts195
Revert "[ML] Correct query times for model plot and forecast (#329)"
80c23ba 
This reverts commit c53a6f0.
@droberts195 droberts195 mentioned this pull request Dec 6, 2018
Merged
droberts195 added a commit that referenced this pull request Dec 6, 2018
@droberts195
Revert "[ML] Correct query times for model plot and forecast (#329)" (#…
f418a70 
…330)

This reverts commit c53a6f0.

The change that was made in #329 changes the rule that all ML
results had timestamps that were the start time of the bucket
they relate to. I don't think this change should be made in a
patch. Further investigation of the side effects is required.
@tveasey tveasey changed the title [ML] Correct query times for model plot and forecast [6.5.3][ML] Correct query times for model plot and forecast Dec 10, 2018
@tveasey tveasey deleted the port/327_6_5 branch December 13, 2018 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport v6.5.3
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tveasey