HeartSaVioR commented on a change in pull request #25943:
[WIP][SPARK-29261][SQL][CORE] Support recover live entities from KVStore for
(SQL)AppStatusListener
URL: https://github.com/apache/spark/pull/25943#discussion_r331909508
##########
File path: core/src/main/scala/org/apache/spark/status/LiveEntity.scala
##########
@@ -59,21 +59,23 @@ private[spark] abstract class LiveEntity {
}
-private class LiveJob(
+private[spark] class LiveJob(
val jobId: Int,
- name: String,
+ val name: String,
val submissionTime: Option[Date],
val stageIds: Seq[Int],
- jobGroup: Option[String],
- numTasks: Int,
- sqlExecutionId: Option[Long]) extends LiveEntity {
+ val jobGroup: Option[String],
+ val numTasks: Int,
+ val sqlExecutionId: Option[Long]) extends LiveEntity {
var activeTasks = 0
var completedTasks = 0
var failedTasks = 0
// Holds both the stage ID and the task index, packed into a single long
value.
val completedIndices = new OpenHashSet[Long]()
+ // will only be set when recover LiveJob is needed.
Review comment:
Could we add `completedIndices` and `completedStages` into new KVStore
entity class, and restore LiveJob from both JobDataWrapper and new class? I
guess you would like to avoid modifying JobDataWrapper, but I'd worry the state
of instance differs from origin vs restored so would like to explore new option
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
