[
https://issues.apache.org/jira/browse/FLINK-4609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15504048#comment-15504048
]
ASF GitHub Bot commented on FLINK-4609:
---------------------------------------
Github user apivovarov commented on a diff in the pull request:
https://github.com/apache/flink/pull/2490#discussion_r79441284
--- Diff:
flink-java/src/test/java/org/apache/flink/api/java/sca/UdfAnalyzerTest.java ---
@@ -971,10 +971,7 @@ public void
testForwardWithAtLeastOneIterationAssumptionForJavac() {
public void reduce(Iterable<Tuple2<Long, Long>> values,
Collector<Long> out) throws Exception {
Long id = 0L;
for (Tuple2<Long, Long> value : values) {
- id = value.f0;
--- End diff --
Just reverted this change. Thank you!
> Remove redundant check for null in CrossOperator
> ------------------------------------------------
>
> Key: FLINK-4609
> URL: https://issues.apache.org/jira/browse/FLINK-4609
> Project: Flink
> Issue Type: Bug
> Components: Java API
> Affects Versions: 1.1.2
> Reporter: Alexander Pivovarov
> Priority: Trivial
>
> CrossOperator checks input1 and input2 for null after they were dereferenced
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
