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

Fix NPE when streaming commit stats #11266

Merged
merged 1 commit into from May 21, 2015
Merged

Fix NPE when streaming commit stats #11266

merged 1 commit into from May 21, 2015

Conversation

brwe
Copy link
Contributor

@brwe brwe commented May 20, 2015

commit id is only written from lucene 5 on and therefore can be null.

@brwe
fix NPE when streaming commit stats
4bdf8aa 
commit id is only written from lucene 5 on
@martijnvg
Copy link
Member

LGTM

@s1monw
Copy link
Contributor

s1monw commented May 20, 2015

LGTM too

brwe added a commit that referenced this pull request May 21, 2015
@brwe
Merge pull request #11266 from brwe/npe-commitstats
17eec9d 
fix NPE when streaming commit stats
@brwe brwe merged commit 17eec9d into elastic:master May 21, 2015
@s1monw
Copy link
Contributor

s1monw commented May 21, 2015

@brwe I think you can remove the 1.6 label this stuff is not in 1.x no?

@s1monw s1monw removed the v1.6.0 label May 22, 2015
@clintongormley clintongormley changed the title fix NPE when streaming commit stats Fix NPE when streaming commit stats Jun 8, 2015
@clintongormley clintongormley added :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. and removed :Engine :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. labels Feb 13, 2018
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
>bug :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@brwe @martijnvg @s1monw @clintongormley