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

    https://github.com/apache/flink/pull/5573#discussion_r175735982
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/json/SerializedValueDeserializer.java
 ---
    @@ -34,6 +36,10 @@
     
        private static final long serialVersionUID = 1L;
     
    +   public SerializedValueDeserializer() {
    --- End diff --
    
    Why do we add this default constructor? I prefer to specify the type 
explicitly. Also if it is a simple `TypeReference<SerializedValue<Object>>`.


---

Reply via email to