Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

Today we still support reading store corruption markers of versions that
haven't been written since 1.7. This commit removes this legacy support.

Today we still support reading store corruption markers of versions that
haven't been written since 1.7. This commit removes this legacy support.
@DaveCTurner DaveCTurner added >non-issue :Distributed Indexing/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. v8.0.0 v7.6.0 labels Nov 5, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Store)

Copy link
Contributor

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


import static java.util.Collections.unmodifiableMap;
import static org.elasticsearch.test.VersionUtils.randomVersion;
import static org.hamcrest.Matchers.allOf;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is breaking check style :)

@DaveCTurner DaveCTurner merged commit e0469a7 into elastic:master Nov 5, 2019
@DaveCTurner DaveCTurner deleted the 2019-11-04-remove-support-for-ancient-corrupted-markers branch November 5, 2019 23:26
DaveCTurner added a commit that referenced this pull request Nov 12, 2019
Today we still support reading store corruption markers of versions that
haven't been written since 1.7. This commit removes this legacy support.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. >non-issue v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants