You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hop.apache.org by GitBox <gi...@apache.org> on 2021/01/30 12:22:00 UTC

[GitHub] [incubator-hop] nadment opened a new pull request #589: HOP-2479 Remove IPluginFolder

nadment opened a new pull request #589:
URL: https://github.com/apache/incubator-hop/pull/589


   This changes breaks APIs
   
   - Move the construction of the plugin folders list to the JarCache class
   - Remove the need to call populateFolders() in each PluginType 
   - Remove methods populateFolders(), getPath(), isReturn() in BasePluginType 
   - Clean JarCache class
   - Add HOP_PLUGIN_BASE_FOLDERS,HOP_SHARED_JDBC_FOLDER to AboutDialog
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-hop] hansva commented on pull request #589: HOP-2479 Remove IPluginFolder

Posted by GitBox <gi...@apache.org>.
hansva commented on pull request #589:
URL: https://github.com/apache/incubator-hop/pull/589#issuecomment-771090393


   ```
   WARNING: An illegal reflective access operation has occurred
   WARNING: Illegal reflective access by org.apache.hop.core.plugins.HopURLClassLoader (file:/Users/hans/git/incubator-hop/assemblies/client/target/hop/lib/hop-core-0.70-SNAPSHOT.jar) to field java.net.URLClassLoader.ucp
   WARNING: Please consider reporting this to the maintainers of org.apache.hop.core.plugins.HopURLClassLoader
   WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
   WARNING: All illegal access operations will be denied in a future release
   ```


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-hop] hansva merged pull request #589: HOP-2479 Remove IPluginFolder

Posted by GitBox <gi...@apache.org>.
hansva merged pull request #589:
URL: https://github.com/apache/incubator-hop/pull/589


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-hop] hansva edited a comment on pull request #589: HOP-2479 Remove IPluginFolder

Posted by GitBox <gi...@apache.org>.
hansva edited a comment on pull request #589:
URL: https://github.com/apache/incubator-hop/pull/589#issuecomment-771090393


   
   
   ```
   WARNING: An illegal reflective access operation has occurred
   WARNING: Illegal reflective access by org.apache.hop.core.plugins.HopURLClassLoader (file:/Users/hans/git/incubator-hop/assemblies/client/target/hop/lib/hop-core-0.70-SNAPSHOT.jar) to field java.net.URLClassLoader.ucp
   WARNING: Please consider reporting this to the maintainers of org.apache.hop.core.plugins.HopURLClassLoader
   WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
   WARNING: All illegal access operations will be denied in a future release
   ```


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-hop] hansva commented on pull request #589: HOP-2479 Remove IPluginFolder

Posted by GitBox <gi...@apache.org>.
hansva commented on pull request #589:
URL: https://github.com/apache/incubator-hop/pull/589#issuecomment-771091991


   Never mind, was running in Java 11


----------------------------------------------------------------
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:
users@infra.apache.org