Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/5970#discussion_r187797400
--- Diff:
flink-java/src/test/java/org/apache/flink/api/java/sca/UdfAnalyzerTest.java ---
@@ -62,6 +62,10 @@
@SuppressWarnings("serial")
public class UdfAnalyzerTest {
+ private static TypeInformation<Tuple2<String, Integer>>
stringIntTuple2TypeInfo = TypeInformation.of(new TypeHint<Tuple2<String,
Integer>>(){});
--- End diff --
These fields are constants, so they should be final. Can you add the
modified and rename the fields to match the naming convention?
---
