Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/19559#discussion_r147068227
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -844,7 +863,12 @@ class Analyzer(
q.transformExpressionsUp {
case u @ UnresolvedAttribute(nameParts) =>
// Leave unchanged if resolution fails. Hopefully will be
resolved next round.
- val result = withPosition(u) { q.resolveChildren(nameParts,
resolver).getOrElse(u) }
+ val result =
+ withPosition(u) {
+ q.resolveChildren(nameParts, resolver).getOrElse {
--- End diff --
You can also use `orElse`:
`q.resolveChildren(nameParts,
resolver).orElse(resolveAsLiteralFunctions(nameParts)).getOrElse(u)`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
