This is an automated email from the ASF dual-hosted git repository.
srowen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 85581f6 [SPARK-33925][CORE][FOLLOW-UP] Remove the unused variables
'secMgr'
85581f6 is described below
commit 85581f6dac5c116d3d83101044cc3a85bfca41bf
Author: PengLei <[email protected]>
AuthorDate: Mon Mar 22 12:02:25 2021 -0500
[SPARK-33925][CORE][FOLLOW-UP] Remove the unused variables 'secMgr'
### What changes were proposed in this pull request?
Remove the unused variable 'secMgr' in SparkSubmit.scala and
DriverWrapper.scala
In jira https://issues.apache.org/jira/browse/SPARK-33925, The last usage
of SecurityManager in Utils.fetchFile was removed. We don't need the variable
anymore
### Why are the changes needed?
For better readablity of codes
### Does this PR introduce _any_ user-facing change?
No,dev-only
### How was this patch tested?
Manually complied. Github Actions and Jenkins build should test it out as
well.
Closes #31928 from Peng-Lei/rm_secMgr.
Authored-by: PengLei <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
---
core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala | 1 -
core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala | 1 -
2 files changed, 2 deletions(-)
diff --git a/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala
b/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala
index 2a89078..e5fd027 100644
--- a/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala
@@ -366,7 +366,6 @@ private[spark] class SparkSubmit extends Logging {
args.pyFiles = Option(args.pyFiles).map(resolveGlobPaths(_,
hadoopConf)).orNull
args.archives = Option(args.archives).map(resolveGlobPaths(_,
hadoopConf)).orNull
- lazy val secMgr = new SecurityManager(sparkConf)
// In client mode, download remote files.
var localPrimaryResource: String = null
diff --git
a/core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala
b/core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala
index 61fb929..9176897 100644
--- a/core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/worker/DriverWrapper.scala
@@ -74,7 +74,6 @@ object DriverWrapper extends Logging {
private def setupDependencies(loader: MutableURLClassLoader, userJar:
String): Unit = {
val sparkConf = new SparkConf()
- val secMgr = new SecurityManager(sparkConf)
val hadoopConf = SparkHadoopUtil.newConfiguration(sparkConf)
val ivyProperties = DependencyUtils.getIvyProperties()
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
