[
https://issues.apache.org/jira/browse/FLINK-2114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14564840#comment-14564840
]
ASF GitHub Bot commented on FLINK-2114:
---------------------------------------
GitHub user ggevay opened a pull request:
https://github.com/apache/flink/pull/748
[FLINK-2114] [streaming] Fix null check in PunctuationPolicy.toString()
Added missing parenthesis, and added a call to PunctuationPolicy.toString()
in a test.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ggevay/flink punctuationPolicyFix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/748.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #748
----
commit 6033e4d23068ebe812ad817609c40f18d1f9bb2b
Author: Gabor Gevay <[email protected]>
Date: 2015-05-26T01:55:03Z
[FLINK-2114] [streaming] Fix null check in PunctuationPolicy.toString()
----
> PunctuationPolicy.toString() throws NullPointerException if extractor is null
> -----------------------------------------------------------------------------
>
> Key: FLINK-2114
> URL: https://issues.apache.org/jira/browse/FLINK-2114
> Project: Flink
> Issue Type: Bug
> Components: Streaming
> Reporter: Gabor Gevay
> Assignee: Gabor Gevay
> Priority: Minor
>
> Parenthesis is missing in PunctuationPolicy.toString() around the conditional
> operator checking for not null, which makes the condition always true.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
