Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22429#discussion_r219729210
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
 ---
    @@ -668,11 +670,19 @@ case class WholeStageCodegenExec(child: 
SparkPlan)(val codegenStageId: Int)
       override def generateTreeString(
           depth: Int,
           lastChildren: Seq[Boolean],
    -      builder: StringBuilder,
    +      writer: Writer,
           verbose: Boolean,
           prefix: String = "",
    -      addSuffix: Boolean = false): StringBuilder = {
    -    child.generateTreeString(depth, lastChildren, builder, verbose, 
s"*($codegenStageId) ")
    +      addSuffix: Boolean = false,
    +      maxFields: Option[Int]): Unit = {
    +    child.generateTreeString(
    +      depth,
    +      lastChildren,
    +      writer,
    +      verbose,
    +      s"*($codegenStageId) ",
    +      false,
    --- End diff --
    
    named boolean : `addSuffix = false`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to