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

UIMA AS client should not create JNDI InitialContext

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.4.0AS
    • Async Scaleout
    • 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

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            schor Marshall Schor
            cwiklik Jaroslaw Cwiklik
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment