Uploaded image for project: 'UIMA'
  1. UIMA
  2. UIMA-2249

UIMA AS client should not create JNDI InitialContext

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.4.0AS
    • Component/s: Async Scaleout
    • Labels:
      None

      Description

      In BaseUIMAAsynchronousEngine_impl.initialize() method the following lines of code should be removed:

      Properties props = new Properties();
      props.setProperty(Context.INITIAL_CONTEXT_FACTORY,"org.apache.activemq.jndi.ActiveMQInitialContextFactory");
      props.setProperty(Context.PROVIDER_URL,brokerURI);
      jndiContext = new InitialContext(props);

      This is dead code not used for anything. Marshall, actually reported seeing NPE thrown from "new InitialContext(props)" when he attached YourKit profiler. Not exactly sure why these NPEs were thrown, since this is not happening when UIMA AS client runs normally ( not under profiler's control). In any case, this code is not needed a should be removed

        Attachments

          Activity

            People

            • Assignee:
              schor Marshall Schor
              Reporter:
              cwiklik Jerry Cwiklik
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: