Skip to content

Commit

Permalink
Follow-up 1142: remove TODO line
Browse files Browse the repository at this point in the history
Signed-off-by: Ahmed Hussein (amahussein) <[email protected]>

Follow-up on #1142 to remove TODO comment line in WholeStageExecParser
  • Loading branch information
amahussein committed Jun 27, 2024
1 parent 823ac09 commit fd4cab2
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ case class WholeStageExecParser(
// average speedup across the execs in the WholeStageCodegen for now
val supportedChildren = childNodes.filterNot(_.shouldRemove)
val avSpeedupFactor = SQLPlanParser.averageSpeedup(supportedChildren.map(_.speedupFactor))
// TODO: revisit the logic behind adding the stages of child nodes to the current node.
// can't rely on the wholeStagecodeGen having a stage if children do so aggregate them together
// for now
val allStagesIncludingChildren = childNodes.flatMap(_.stages).toSet ++ stagesInNode.toSet
Expand Down

0 comments on commit fd4cab2

Please sign in to comment.