bbende commented on a change in pull request #4540:
URL: https://github.com/apache/nifi/pull/4540#discussion_r496726770



##########
File path: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/bootstrap.conf
##########
@@ -66,6 +66,15 @@ java.arg.16=-Djavax.security.auth.useSubjectCredsOnly=true
 # Please see 
https://zookeeper.apache.org/doc/current/zookeeperAdmin.html#sc_adminserver_config
 for configuration options.
 java.arg.17=-Dzookeeper.admin.enableServer=false
 
+# The following options configure a Java Agent to handle native library 
loading.
+# It is needed when a custom jar (eg. JDBC driver) has been configured on a 
component in the flow and this custom jar depends on a native library
+# and tries to load it by its absolute path (java.lang.System.load(String 
filename) method call).
+# Use this Java Agent only if you get "Native Library ... already loaded in 
another classloader" errors otherwise!
+#java.arg.18=-javaagent:./lib/aspectjweaver-${aspectj.version}.jar

Review comment:
       Do we have any concerns about aspectjweaver being directly in the lib 
directory which puts in on the classpath of every single NAR? Wondering if it 
should be separated like `lib/aspectj/` which would not be added to NiFi's 
normal classpath, but I don't know enough to say if it still works correctly 
then.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to