Skip to content

Commit

Permalink
[DOC] Fix comment on SparkPlanGraphEdge
Browse files Browse the repository at this point in the history
## 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 #22268 from juliuszsompolski/sparkplangraphedgedoc.

Authored-by: Juliusz Sompolski <julek@databricks.com>
Signed-off-by: Xiao Li <gatorsmile@gmail.com>
  • Loading branch information
juliuszsompolski authored and gatorsmile committed Aug 29, 2018
1 parent 20b7c68 commit 6b1b10c
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -202,7 +202,7 @@ private[ui] class SparkPlanGraphCluster(


/**
* Represent an edge in the SparkPlan tree. `fromId` is the parent node id, and `toId` is the child
* Represent an edge in the SparkPlan tree. `fromId` is the child node id, and `toId` is the parent
* node id.
*/
private[ui] case class SparkPlanGraphEdge(fromId: Long, toId: Long) {
Expand Down

0 comments on commit 6b1b10c

Please sign in to comment.