Github user skambha commented on a diff in the pull request:
https://github.com/apache/spark/pull/17185#discussion_r207027392
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
---
@@ -121,14 +129,14 @@ abstract class Attribute extends LeafExpression with
NamedExpression with NullIn
* @param name The name to be associated with the result of computing
[[child]].
* @param exprId A globally unique id used to check if an
[[AttributeReference]] refers to this
* alias. Auto-assigned if left blank.
- * @param qualifier An optional string that can be used to referred to
this attribute in a fully
- * qualified way. Consider the examples tableName.name,
subQueryAlias.name.
- * tableName and subQueryAlias are possible qualifiers.
+ * @param qualifier An optional Seq of string that can be used to refer to
this attribute in a
+ * fully qualified way. Consider the examples
tableName.name, subQueryAlias.name.
+ * tableName and subQueryAlias are possible qualifiers.
* @param explicitMetadata Explicit metadata associated with this alias
that overwrites child's.
*/
case class Alias(child: Expression, name: String)(
val exprId: ExprId = NamedExpression.newExprId,
- val qualifier: Option[String] = None,
+ val qualifier: Option[Seq[String]] = None,
--- End diff --
I'll look into this.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
