UIMA
  1. UIMA
  2. UIMA-2249

UIMA AS client should not create JNDI InitialContext

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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

        Activity

        Hide
        Marshall Schor added a comment -

        Removed this. I did notice that Yourkit has some special box you can check re: JNDI things, for J2EE, and it was checked... Might have been the underlying cause...

        Show
        Marshall Schor added a comment - Removed this. I did notice that Yourkit has some special box you can check re: JNDI things, for J2EE, and it was checked... Might have been the underlying cause...

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development