[GitHub] flink pull request #6016: [FLINK-8933] Avoid calling Class#newInstance(part ...

[zentol]

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

    https://github.com/apache/flink/pull/6016#discussion_r188246084
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/PojoSerializer.java
 ---
    @@ -193,7 +193,7 @@ public T createInstance() {
                        return null;
                }
                try {
    -                   T t = clazz.newInstance();
    +                   T t = clazz.getDeclaredConstructor().newInstance();
    --- End diff --
    
    this change may have performance implications as noted in the mail thread 
linked in the jira.


---