Skip to content

Fix NPE on Stale Index in IndicesService (#38891) #41012

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
Merged
merged 1 commit into from
Apr 9, 2019
Merged

Fix NPE on Stale Index in IndicesService (#38891) #41012

merged 1 commit into from
Apr 9, 2019

Conversation

original-brownbear
Copy link
Member

@original-brownbear original-brownbear added >bug :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. backport labels Apr 9, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@original-brownbear
Copy link
Member Author

Jenkins run elasticsearch-ci/1

@original-brownbear original-brownbear merged commit e9dfb5b into elastic:6.7 Apr 9, 2019
@original-brownbear original-brownbear deleted the 38891-6.7 branch April 9, 2019 21:05
@howardhuanghua
Copy link
Contributor

howardhuanghua commented Jul 27, 2019 •
edited
Loading

Hi @original-brownbear , I checked the fix for NPE, it's only return null without handling residual index segment files, so somewhere would handle these files in tombstones? Although index meta data is empty, but segment files may still exist.

Thanks,
Howard

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 >bug :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@original-brownbear @elasticmachine @howardhuanghua