Github user advancedxy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20082#discussion_r158699877
--- Diff: core/src/main/scala/org/apache/spark/TaskContext.scala ---
@@ -150,6 +150,10 @@ abstract class TaskContext extends Serializable {
*/
def stageId(): Int
+ /**
+ * The attempt ID of the stage that this task belongs to.
--- End diff --
Of course, I can add more doc.
But is it necessary? I think the field(function) name is already
self-explanatory.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
