mridulm commented on code in PR #43954:
URL: https://github.com/apache/spark/pull/43954#discussion_r1422054329
##########
core/src/main/scala/org/apache/spark/internal/config/package.scala:
##########
@@ -2603,4 +2603,14 @@ package object config {
.stringConf
.toSequence
.createWithDefault("org.apache.spark.sql.connect.client" :: Nil)
+
+ private[spark] val LEGACY_ABORT_STAGE_AFTER_CANCEL_TASKS =
+ ConfigBuilder("spark.scheduler.stage.legacyAbortAfterCancelTasks")
+ .doc("Whether to abort a stage after TaskScheduler.cancelTasks(). This
is used to restore " +
+ "the original behavior in case there are any regressions after abort
stage is removed " +
+ "from TaskScheduler.cancelTasks()")
+ .version("4.0.0")
+ .internal()
+ .booleanConf
+ .createWithDefault(true)
Review Comment:
Given this is a new change, might be good to be conservative about it.
We can change default in 4.1 or later, it does not need to be at major
version.
--
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
