Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22788#discussion_r227202767
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -99,7 +99,7 @@ case class UnresolvedTableValuedFunction(
case class UnresolvedAttribute(nameParts: Seq[String]) extends Attribute
with Unevaluable {
def name: String =
- nameParts.map(n => if (n.contains(".")) s"`$n`" else n).mkString(".")
+ nameParts.map(n => if (nameParts.length > 1 || n.contains("."))
s"`$n`" else n).mkString(".")
--- End diff --
I don't think this is better for `name`, we should update `sql` though.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
