Uploaded image for project: 'OpenEJB'
  1. OpenEJB
  2. OPENEJB-280

Corba classes should be in their own jar

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.2
    • 2.2
    • corba
    • None

    Description

      The corba classes in openejb-core should be in their own jar. Then that jar can be a dependency of both the yoko and sun corba integrations. Currently yoko has to be included in the same classloader as openejb-core so that UtilDelegateImpl can load its delegate using its own classloader:

      public UtilDelegateImpl() throws ClassNotFoundException, IllegalAccessException, InstantiationException {
      String value = System.getProperty(DELEGATE_NAME);
      if (value == null)

      { log.error("No delegate specfied via " + DELEGATE_NAME); throw new IllegalStateException("The property " + DELEGATE_NAME + " must be defined!"); }

      if (log.isDebugEnabled()) log.debug("Set delegate " + value);
      delegate = (UtilDelegate) Class.forName(value).newInstance();
      }

      We might get some traction by using the TCCL here but I think it would be more reliable and better structure to separate the corba classes into another jar.

      Attachments

        Activity

          People

            djencks David Jencks
            djencks David Jencks
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: