[ 
https://issues.apache.org/jira/browse/BEAM-3845?focusedWorklogId=317588&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-317588
 ]

ASF GitHub Bot logged work on BEAM-3845:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 24/Sep/19 15:33
            Start Date: 24/Sep/19 15:33
    Worklog Time Spent: 10m 
      Work Description: lgajowy commented on issue #9613: [BEAM-3845] Remove 
deprecated Class.newInstance() method usage
URL: https://github.com/apache/beam/pull/9613#issuecomment-534614223
 
 
   @alanmyrvold and @yifanzou: we noticed together with @iemejia that this 
class (`TestScripts`) is used only by the `quickstart-java-*.groovy`, 
`starter-generation.groovy` and `mobilegaming-java-*.groovy` scripts. The 
scripts (quickstart, mobilegaming...) in turn seem to be not used by any job or 
gradle task in Beam's repo.  
   Therefore, I have a question: are the groovy scripts still used anywhere or 
will be used? Or is it a "dead code" and we should delete this? 
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 317588)
    Time Spent: 40m  (was: 0.5h)

> Avoid calling Class#newInstance
> -------------------------------
>
>                 Key: BEAM-3845
>                 URL: https://issues.apache.org/jira/browse/BEAM-3845
>             Project: Beam
>          Issue Type: Task
>          Components: sdk-java-core
>            Reporter: Ted Yu
>            Assignee: Lukasz Gajowy
>            Priority: Minor
>              Labels: triaged
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> 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
(v8.3.4#803005)

Reply via email to