Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/20226#discussion_r162224065
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -45,7 +46,12 @@ trait CodegenSupport extends SparkPlan {
case _: SortMergeJoinExec => "smj"
case _: RDDScanExec => "rdd"
case _: DataSourceScanExec => "scan"
- case _ => nodeName.toLowerCase(Locale.ROOT)
--- End diff --
This caused one of the tests to fail as the nodename generated was not a
single word (like before) but something like `scan in-memory my_table`... which
does not compile with codegen. The change done was to retain only the
alpha-numeric characters in the `nodeName`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
