Skip to content

Conversation

@vladimirdolzhenko
Copy link
Contributor

@vladimirdolzhenko vladimirdolzhenko commented Sep 20, 2018

CorruptIndexException, IndexFormatTooOldException and IndexFormatTooNewException are handled in the similar way at Store#getMetadata. It is worth to do same in Store#readLastCommittedSegmentsInfo

Closes #33916

@vladimirdolzhenko vladimirdolzhenko added the :Distributed Indexing/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. label Sep 20, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@vladimirdolzhenko vladimirdolzhenko changed the title keep IndexFormatToo(OldException and IndexFormatTooNewException… keep IndexFormatToo(Old|New)Exception in corruption marker Sep 20, 2018
@vladimirdolzhenko vladimirdolzhenko merged commit 9c03168 into elastic:master Sep 21, 2018
@vladimirdolzhenko
Copy link
Contributor Author

thanks @ywelsch

@vladimirdolzhenko vladimirdolzhenko deleted the fix/33916 branch September 21, 2018 12:00
vladimirdolzhenko added a commit that referenced this pull request Sep 21, 2018
… in corruption marker (#33920)

Closes #33916

(cherry picked from commit 9c03168)
@vladimirdolzhenko vladimirdolzhenko added v7.0.0 v6.5.0 >test Issues or PRs that are addressing/adding tests labels Sep 21, 2018
kcm pushed a commit that referenced this pull request Oct 30, 2018
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. >test Issues or PRs that are addressing/adding tests v6.5.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants