Skip to content

Commit 6b1b10c

Browse files
juliuszsompolskigatorsmile
authored andcommitted
[DOC] Fix comment on SparkPlanGraphEdge
## What changes were proposed in this pull request? `fromId` is the child, and `toId` is the parent, see line 127 in `buildSparkPlanGraphNode` above. The edges in Spark UI also go from child to parent. ## How was this patch tested? Comment change only. Inspected code above. Inspected how the edges in Spark UI look like. Closes apache#22268 from juliuszsompolski/sparkplangraphedgedoc. Authored-by: Juliusz Sompolski <[email protected]> Signed-off-by: Xiao Li <[email protected]>
1 parent 20b7c68 commit 6b1b10c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/ui/SparkPlanGraph.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ private[ui] class SparkPlanGraphCluster(
202202

203203

204204
/**
205-
* Represent an edge in the SparkPlan tree. `fromId` is the parent node id, and `toId` is the child
205+
* Represent an edge in the SparkPlan tree. `fromId` is the child node id, and `toId` is the parent
206206
* node id.
207207
*/
208208
private[ui] case class SparkPlanGraphEdge(fromId: Long, toId: Long) {

0 commit comments

Comments
 (0)