[GitHub] [skywalking-java] rechardguo commented on a diff in pull request #527: Support agent running in the JBoss relative with its ModuleClassloader.

["rechardguo (via GitHub)" <gi...@apache.org>]

rechardguo commented on code in PR #527:
URL: https://github.com/apache/skywalking-java/pull/527#discussion_r1188646875


##########
apm-sniffer/apm-agent-core/src/main/java/org/apache/skywalking/apm/agent/core/boot/AgentPackagePath.java:
##########
@@ -50,6 +50,9 @@ private static File findPath() throws AgentPackageNotFoundException {
         String classResourcePath = AgentPackagePath.class.getName().replaceAll("\\.", "/") + ".class";
 
         URL resource = ClassLoader.getSystemClassLoader().getResource(classResourcePath);
+        if (resource == null) {
+            resource = AgentPackagePath.class.getClassLoader().getResource(classResourcePath);
+        }

Review Comment:
   Actually i want to change as that. But to compatible with the old funtion, I am not quite sure the very beginning design. 



-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org