Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/5970#discussion_r187589977
--- Diff:
flink-core/src/test/java/org/apache/flink/api/common/typeinfo/TypeInformationTest.java
---
@@ -61,7 +61,7 @@ public void testOfGenericClassForGenericType() {
@Test
public void testOfTypeHint() {
assertEquals(BasicTypeInfo.STRING_TYPE_INFO,
TypeInformation.of(String.class));
- assertEquals(BasicTypeInfo.STRING_TYPE_INFO,
TypeInformation.of(new TypeHint<String>(){}));
+ assertEquals(BasicTypeInfo.STRING_TYPE_INFO, Types.STRING);
--- End diff --
Please undo this one, the test explicitly tests the TypeHint use.
---