You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2023/06/26 17:06:29 UTC

[spark] branch master updated: [SPARK-44188][CORE] Remove useless `resetAllPartitions` method in `ActiveJob`

This is an automated email from the ASF dual-hosted git repository.

dongjoon 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 e96687cc847 [SPARK-44188][CORE] Remove useless `resetAllPartitions` method in `ActiveJob`
e96687cc847 is described below

commit e96687cc84709b2e057af79433df68b244f95bdc
Author: Jia Fan <fa...@qq.com>
AuthorDate: Mon Jun 26 10:06:14 2023 -0700

    [SPARK-44188][CORE] Remove useless `resetAllPartitions` method in `ActiveJob`
    
    ### What changes were proposed in this pull request?
    Remove useless `resetAllPartitions` method in `ActiveJob`. It unused when #22158.
    
    ### Why are the changes needed?
    Clean code
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    Unnecessary
    
    Closes #41737 from Hisoka-X/SPARK-44188_remove_activejob_method.
    
    Authored-by: Jia Fan <fa...@qq.com>
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
 core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala b/core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala
index 89f0c05c815..9876668194a 100644
--- a/core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala
+++ b/core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala
@@ -63,10 +63,4 @@ private[spark] class ActiveJob(
   val finished = Array.fill[Boolean](numPartitions)(false)
 
   var numFinished = 0
-
-  /** Resets the status of all partitions in this stage so they are marked as not finished. */
-  def resetAllPartitions(): Unit = {
-    (0 until numPartitions).foreach(finished.update(_, false))
-    numFinished = 0
-  }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org