Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/20226#discussion_r161036084
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveTableScanExec.scala
---
@@ -62,6 +62,8 @@ case class HiveTableScanExec(
override def conf: SQLConf = sparkSession.sessionState.conf
+ override def nodeName: String = s"${super.nodeName}
(${relation.tableMeta.qualifiedName})"
--- End diff --
[DataSourceScanExec](https://github.com/apache/spark/blob/ee56fc3432e7e328c29b88a255f7e2c2a4739754/sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala#L47)
is using `unquotedString` and [so does this
PR](https://github.com/apache/spark/blob/da35574297d96eb750bdfee755a48defc36e284a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala#L265).
Rest implementations of `LeafExecNode` do not override `nodeName` or depend on
base class (ie. `TreeNode`)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
