Skip to content

[FLINK-5276] [eg] Check for null when archiving prior execution attempts#2957

Closed
uce wants to merge 1 commit intoapache:release-1.1from
uce:5276-archive_execvertex_npe
Closed

[FLINK-5276] [eg] Check for null when archiving prior execution attempts#2957
uce wants to merge 1 commit intoapache:release-1.1from
uce:5276-archive_execvertex_npe

Conversation

@uce
Copy link
Copy Markdown
Contributor

@uce uce commented Dec 7, 2016

The EvictingBoundedList returns the default value for all evicted elements. This could lead to NPE when archiving an ExecutionVertex.

/cc @StefanRRichter

@StefanRRichter
Copy link
Copy Markdown
Contributor

LGTM +1

asfgit pushed a commit that referenced this pull request Dec 7, 2016
@uce uce closed this Dec 7, 2016
skidder pushed a commit to muxinc/flink that referenced this pull request Dec 27, 2016
@uce uce deleted the 5276-archive_execvertex_npe branch February 16, 2017 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants