[
https://issues.apache.org/jira/browse/FLINK-8933?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16476048#comment-16476048
]
ASF GitHub Bot commented on FLINK-8933:
---------------------------------------
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/6016
OK, I agree @StephanEwen , let's give it a low priority. Actually, I know
this big change can bring nondeterminacy to the codebase. So when I assigned
this issue to myself, I asked your opinion in JIRA. But it does not matter, the
discussion is significative.
Thanks @zentol started the discussion and preciseness about the change. I
think we could discuss again about this issue, as I said before, google's
error-prone also marked this as a bug pattern and JDK team is also preciseness
before they gave this statement.
> Avoid calling Class#newInstance
> -------------------------------
>
> Key: FLINK-8933
> URL: https://issues.apache.org/jira/browse/FLINK-8933
> Project: Flink
> Issue Type: Task
> Reporter: Ted Yu
> Assignee: vinoyang
> Priority: Minor
>
> Class#newInstance is deprecated starting in Java 9 -
> https://bugs.openjdk.java.net/browse/JDK-6850612 - because it may throw
> undeclared checked exceptions.
> The suggested replacement is getDeclaredConstructor().newInstance(), which
> wraps the checked exceptions in InvocationException.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
