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_r331842134
##########
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.
+ var completedIndicesNum = 0
Review comment:
Could we follow the name style from JobData? They have fields for number of
XXX as `numXXX`.
----------------------------------------------------------------
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]
