You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Donald Woods (JIRA)" <ji...@apache.org> on 2009/10/28 20:01:59 UTC

[jira] Updated: (GERONIMO-4936) OpenJPA2 plugin needs to replace use of Configuration.getConfigurationClassLoader()

     [ https://issues.apache.org/jira/browse/GERONIMO-4936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Donald Woods updated GERONIMO-4936:
-----------------------------------

    Description: 
As pointed out by the TODO added in r830529 by Rick McGuire -

plugins/openjpa2/geronimo-openjpa2/src/main/java/org/apache/geronimo/openjpa/ConfigurationMonitorGBean.java
{code}
     private void configurationRunning(AbstractName name) {
         try {
             Configuration config = (Configuration)kernel.getGBean(name);
-            classLoaderMap.put(name, config.getConfigurationClassLoader());
+// TODO:  This needs to be resolved as to what class loader should be registered
+//          classLoaderMap.put(name, config.getConfigurationClassLoader());
         } catch (GBeanNotFoundException gnfe) {
             log.warn("Could not retrieve GBean for artifact: " + name.toString(), gnfe);
         }
     }
{code}

  was:
As pointed out by the TODO added in r830529 by Rick McGuire -

plugins/openjpa2/geronimo-openjpa2/src/main/java/org/apache/geronimo/openjpa/ConfigurationMonitorGBean.java

     private void configurationRunning(AbstractName name) {
         try {
             Configuration config = (Configuration)kernel.getGBean(name);
-            classLoaderMap.put(name, config.getConfigurationClassLoader());
+// TODO:  This needs to be resolved as to what class loader should be registered
+//          classLoaderMap.put(name, config.getConfigurationClassLoader());
         } catch (GBeanNotFoundException gnfe) {
             log.warn("Could not retrieve GBean for artifact: " + name.toString(), gnfe);
         }
     }



> OpenJPA2 plugin needs to replace use of Configuration.getConfigurationClassLoader()
> -----------------------------------------------------------------------------------
>
>                 Key: GERONIMO-4936
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4936
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: osgi
>    Affects Versions: 3.0
>            Reporter: Donald Woods
>             Fix For: 3.0
>
>
> As pointed out by the TODO added in r830529 by Rick McGuire -
> plugins/openjpa2/geronimo-openjpa2/src/main/java/org/apache/geronimo/openjpa/ConfigurationMonitorGBean.java
> {code}
>      private void configurationRunning(AbstractName name) {
>          try {
>              Configuration config = (Configuration)kernel.getGBean(name);
> -            classLoaderMap.put(name, config.getConfigurationClassLoader());
> +// TODO:  This needs to be resolved as to what class loader should be registered
> +//          classLoaderMap.put(name, config.getConfigurationClassLoader());
>          } catch (GBeanNotFoundException gnfe) {
>              log.warn("Could not retrieve GBean for artifact: " + name.toString(), gnfe);
>          }
>      }
> {code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.