Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20226#discussion_r166193203
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExistingRDD.scala ---
@@ -103,6 +103,8 @@ case class ExternalRDDScanExec[T](
override lazy val metrics = Map(
"numOutputRows" -> SQLMetrics.createMetric(sparkContext, "number of
output rows"))
+ override val nodeName: String = s"Scan ExternalRDD
${output.map(_.name).mkString("[", ",", "]")}"
--- End diff --
I don't think including the output in the node name is a good idea.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
