[spark] branch master updated: [SPARK-33925][CORE][FOLLOW-UP] Remove the unused variables 'secMgr'

[srowen]

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 <18066542...@189.cn>
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 <18066542...@189.cn>
    Signed-off-by: Sean Owen <sro...@gmail.com>
---
 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: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org