Skip to content

Commit 79b10a1

Browse files
iRaksondongjoon-hyun
authored andcommitted
[SPARK-28929][CORE] Spark Logging level should be INFO instead of DEBUG in Executor Plugin API
### What changes were proposed in this pull request? Log levels in Executor.scala are changed from DEBUG to INFO. ### Why are the changes needed? Logging level DEBUG is too low here. These messages are simple acknowledgement for successful loading and initialization of plugins. So its better to keep them in INFO level. ### Does this PR introduce any user-facing change? No ### How was this patch tested? Manually tested. Closes #25634 from iRakson/ExecutorPlugin. Authored-by: iRakson <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent db996cc commit 79b10a1

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

core/src/main/scala/org/apache/spark/executor/Executor.scala

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -139,20 +139,20 @@ private[spark] class Executor(
139139
private val executorPlugins: Seq[ExecutorPlugin] = {
140140
val pluginNames = conf.get(EXECUTOR_PLUGINS)
141141
if (pluginNames.nonEmpty) {
142-
logDebug(s"Initializing the following plugins: ${pluginNames.mkString(", ")}")
142+
logInfo(s"Initializing the following plugins: ${pluginNames.mkString(", ")}")
143143

144144
// Plugins need to load using a class loader that includes the executor's user classpath
145145
val pluginList: Seq[ExecutorPlugin] =
146146
Utils.withContextClassLoader(replClassLoader) {
147147
val plugins = Utils.loadExtensions(classOf[ExecutorPlugin], pluginNames, conf)
148148
plugins.foreach { plugin =>
149149
plugin.init()
150-
logDebug(s"Successfully loaded plugin " + plugin.getClass().getCanonicalName())
150+
logInfo(s"Successfully loaded plugin " + plugin.getClass().getCanonicalName())
151151
}
152152
plugins
153153
}
154154

155-
logDebug("Finished initializing plugins")
155+
logInfo("Finished initializing plugins")
156156
pluginList
157157
} else {
158158
Nil

0 commit comments

Comments
 (0)