Ngone51 commented on code in PR #39459:
URL: https://github.com/apache/spark/pull/39459#discussion_r1071178376
##########
core/src/test/scala/org/apache/spark/AccumulatorSuite.scala:
##########
@@ -89,6 +89,38 @@ class AccumulatorSuite extends SparkFunSuite with Matchers
with LocalSparkContex
assert(AccumulatorContext.get(100000).isEmpty)
}
+
+ test("SPARK-41497: accumulators should be reported in the case of task retry
with rdd cache") {
+ // Set up a cluster with 2 executors
+ val conf = new SparkConf()
+ .setMaster("local-cluster[2, 1,
1024]").setAppName("TaskSchedulerImplSuite")
Review Comment:
The app name is not appropriate here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
