Skip to content

Ensure meta and document field maps are never null in GetResult#50112

Merged
rjernst merged 2 commits intoelastic:masterfrom
rjernst:mappings_cleanup1
Dec 12, 2019
Merged

Ensure meta and document field maps are never null in GetResult#50112
rjernst merged 2 commits intoelastic:masterfrom
rjernst:mappings_cleanup1

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Dec 12, 2019

This commit ensures deseriable a GetResult from StreamInput does not
leave metaFields and documentFields null. This could cause an NPE in
situations where upsert response for a document that did not exist is
passed back to a node that forwarded the upsert request.

closes #48215

This commit ensures deseriable a GetResult from StreamInput does not
leave metaFields and documentFields null. This could cause an NPE in
situations where upsert response for a document that did not exist is
passed back to a node that forwarded the upsert request.

closes elastic#48215
@rjernst rjernst added >bug :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. v8.0.0 v7.6.0 labels Dec 12, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented Dec 12, 2019

@elasticmachine
run elasticsearch-ci/bwc
run elasticsearch-ci/default-distro

Copy link
Copy Markdown
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM

@rjernst rjernst merged commit 03abcb4 into elastic:master Dec 12, 2019
@rjernst rjernst deleted the mappings_cleanup1 branch December 12, 2019 05:31
rjernst added a commit that referenced this pull request Dec 12, 2019
This commit ensures deseriable a GetResult from StreamInput does not
leave metaFields and documentFields null. This could cause an NPE in
situations where upsert response for a document that did not exist is
passed back to a node that forwarded the upsert request.

closes #48215
SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
…tic#50112)

This commit ensures deseriable a GetResult from StreamInput does not
leave metaFields and documentFields null. This could cause an NPE in
situations where upsert response for a document that did not exist is
passed back to a node that forwarded the upsert request.

closes elastic#48215
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scripted Update NPE

4 participants