Skip to content

[ML][Data Frame] enable bwc tests again, adjusting after backport#44720

Merged
benwtrent merged 1 commit intoelastic:masterfrom
benwtrent:tests/ml-df-unmute-bwc
Jul 23, 2019
Merged

[ML][Data Frame] enable bwc tests again, adjusting after backport#44720
benwtrent merged 1 commit intoelastic:masterfrom
benwtrent:tests/ml-df-unmute-bwc

Conversation

@benwtrent
Copy link
Copy Markdown
Member

@benwtrent benwtrent added >test Issues or PRs that are addressing/adding tests v8.0.0 :ml/Transform Transform labels Jul 22, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/ml-core

Copy link
Copy Markdown

@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

iverase pushed a commit to iverase/elasticsearch that referenced this pull request Jul 23, 2019
This commit removes support for the translog checkpoint format from versions
before 6.0.0 since 7.x versions are incompatible with indices from these
versions.

Relates elastic#44720
Fixes elastic#44210
@droberts195
Copy link
Copy Markdown

@benwtrent I just went to mute these same tests for my backport of #44350 and found they were still muted. To save churn, please could you hold off merging this PR until near the end of your day. Then I should be able to get my backport done with the same mutes.

@droberts195
Copy link
Copy Markdown

Actually, ignore the previous comment @benwtrent. My PR is proving more problematic to backport than I thought, so since this PR is green you might as well merge it and then I'll re-mute these tests once I've fixed my backport.

@benwtrent benwtrent merged commit e22ce5a into elastic:master Jul 23, 2019
@benwtrent benwtrent deleted the tests/ml-df-unmute-bwc branch July 23, 2019 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml/Transform Transform >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants