diff --git a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginBootstrap.java b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginBootstrap.java index d1b72047d63f246f155156a33e62675226226bf0..a355603d7e178aad8bbe76d6c31f9a1d28eb89da 100644 --- a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginBootstrap.java +++ b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginBootstrap.java @@ -27,7 +27,7 @@ public class PluginBootstrap { List resources = resolver.getResources(); if (resources == null || resources.size() == 0) { - logger.info("no plugin files (skywalking-plugin.properties) found, continue to start application."); + logger.info("no plugin files (skywalking-plugin.def) found, continue to start application."); return new ArrayList(); } @@ -35,7 +35,7 @@ public class PluginBootstrap { try { PluginCfg.INSTANCE.load(pluginUrl.openStream()); } catch (Throwable t) { - logger.error(t, "plugin [{}] init failure.", pluginUrl); + logger.error(t, "plugin file [{}] init failure.", pluginUrl); } } diff --git a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginCfg.java b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginCfg.java index 3fae43aa53de6fb6c0e143b778cb929dfee57c44..5949342ae407639fb315bf3295b62c6538603d17 100644 --- a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginCfg.java +++ b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginCfg.java @@ -1,14 +1,15 @@ package org.skywalking.apm.agent.core.plugin; +import org.skywalking.apm.agent.core.plugin.exception.IllegalPluginDefineException; +import org.skywalking.apm.logging.ILog; +import org.skywalking.apm.logging.LogManager; + import java.io.BufferedReader; import java.io.IOException; import java.io.InputStream; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.List; -import org.skywalking.apm.agent.core.plugin.exception.IllegalPluginDefineException; -import org.skywalking.apm.logging.ILog; -import org.skywalking.apm.logging.LogManager; public enum PluginCfg { INSTANCE; @@ -31,7 +32,7 @@ public enum PluginCfg { pluginClassList.add(plugin); } } catch (IllegalPluginDefineException e) { - logger.error("Failed to format plugin define.", e); + logger.error(e,"Failed to format plugin({}) define.", pluginDefine); } } } finally { diff --git a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginResourcesResolver.java b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginResourcesResolver.java index b223364374959feca1f5fb61fd8afa01b9d4d639..34e56e93f9814ceab00cb19218c9a9d8914c147d 100644 --- a/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginResourcesResolver.java +++ b/apm-sniffer/apm-agent-core/src/main/java/org/skywalking/apm/agent/core/plugin/PluginResourcesResolver.java @@ -24,10 +24,6 @@ public class PluginResourcesResolver { try { urls = getDefaultClassLoader().getResources("skywalking-plugin.def"); - if (!urls.hasMoreElements()) { - logger.info("no plugin files (skywalking-plugin.def) found"); - } - while (urls.hasMoreElements()) { URL pluginUrl = urls.nextElement(); cfgUrlPaths.add(pluginUrl);