[GitHub] flink pull request #5961: [FLINK-8255][DataSet API, DataStream API] key expr...

[fhueske]

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5961#discussion_r186528901
  
    --- Diff: 
flink-streaming-java/src/test/java/org/apache/flink/streaming/util/typeutils/FieldAccessorTest.java
 ---
    @@ -368,4 +369,23 @@ public void testIllegalBasicType2() {
     
                FieldAccessor<Long, Long> f = 
FieldAccessorFactory.getAccessor(tpeInfo, "foo", null);
        }
    +
    +   /**
    +    * Validates that no ClassCastException happens
    +    * should not fail e.g. like in FLINK-8255.
    +    */
    +   @Test
    +   public void testRowTypeInfo() {
    --- End diff --
    
    This test just validates that a `FieldAccessor` is created. At runtime it 
would fail with a `ClassCastException`.


---