Github user zenfenan commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2653#discussion_r190296123
  
    --- Diff: 
nifi-nar-bundles/nifi-jms-bundle/nifi-jms-processors/src/main/java/org/apache/nifi/jms/cf/JMSConnectionFactoryProvider.java
 ---
    @@ -159,13 +159,15 @@ public void enable(ConfigurationContext context) 
throws InitializationException
                     if (logger.isInfoEnabled()) {
                         logger.info("Configuring " + 
this.getClass().getSimpleName() + " for '"
                                 + 
context.getProperty(CONNECTION_FACTORY_IMPL).evaluateAttributeExpressions().getValue()
 + "' to be connected to '"
    -                            + BROKER_URI + "'");
    +                            + 
context.getProperty(BROKER_URI).evaluateAttributeExpressions().getValue() + 
"'");
                     }
    +
                     // will load user provided libraries/resources on the 
classpath
    -                
Utils.addResourcesToClasspath(context.getProperty(CLIENT_LIB_DIR_PATH).evaluateAttributeExpressions().getValue());
    +                final String clientLibPath = 
context.getProperty(CLIENT_LIB_DIR_PATH).evaluateAttributeExpressions().getValue();
    +                ClassLoader customClassLoader = 
ClassLoaderUtils.getCustomClassLoader(clientLibPath, 
this.getClass().getClassLoader(), null);
    +                
Thread.currentThread().setContextClassLoader(customClassLoader);
    --- End diff --
    
    @markap14 Appreciate if you could take a look at this.


---

Reply via email to