[ 
https://issues.apache.org/jira/browse/IGNITE-6745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16259890#comment-16259890
 ] 

Cergey Chaulin edited comment on IGNITE-6745 at 11/30/17 8:01 PM:
------------------------------------------------------------------

[~ezhuravl],
1. The file is present in the patch. I'll add it to the pull request. 
.. Other points will be fixed.

These changes only refer to runtime (i.e. when running under jdk-9 and to run 
as such appropriate modules should be supplied as "-add-opens" etc. in command 
line, probably we need tests for this also). To build under jdk-9 all the 
issues in IGNITE-6728 should be fixed.


was (Author: cossack5):
1. The file is present in the patch. I'll add it to the pull request. 
.. Other points will be fixed.

These changes only refer to runtime (i.e. when running under jdk-9 and to run 
as such appropriate modules should be supplied as "-add-opens" etc. in command 
line, probably we need tests for this also). To build under jdk-9 all the 
issues in IGNITE-6728 should be fixed.

> Java 9: rework usages of URLClassLoader.getURLs()
> -------------------------------------------------
>
>                 Key: IGNITE-6745
>                 URL: https://issues.apache.org/jira/browse/IGNITE-6745
>             Project: Ignite
>          Issue Type: Task
>          Components: general
>    Affects Versions: 2.3
>            Reporter: Vladimir Ozerov
>            Assignee: Cergey Chaulin
>             Fix For: 2.4
>
>         Attachments: IGNITE-6745.patch
>
>
> We use this method in multiple places:
> 1) {{MessageCodeGenerator}}
> 2) {{BinaryContext}}
> 3) {{ClassesGenerator}}
> 4) {{GridUriDeploymentFileProcessor}}
> The problem is that in Java 9 application class loader is not 
> {{URLClassLoader}}, so we cannot get URLs easily. Instead typically it is 
> {{BuiltinClassLoader}}, which refers to {{URLClassLoader}} in it's internal 
> field {{ucp}}.
> Let's refactor all usages of {{URLClassLoader.getURLs}} to some utility 
> method, which will be able to handle both Java 7/8 and Java 9 (through 
> reflection).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to