[spark] branch master updated: [SPARK-43859][SQL] Override toString in LateralColumnAliasReference

[yumwang]

This is an automated email from the ASF dual-hosted git repository.

yumwang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 6fa910b2ab6 [SPARK-43859][SQL] Override toString in 
LateralColumnAliasReference
6fa910b2ab6 is described below

commit 6fa910b2ab6ac619cda76608a109d69e71a8c09d
Author: Yuming Wang <yumw...@ebay.com>
AuthorDate: Wed May 31 13:11:59 2023 +0800

    [SPARK-43859][SQL] Override toString in LateralColumnAliasReference
    
    ### What changes were proposed in this pull request?
    
    This PR makes it override `toString` in `LateralColumnAliasReference`.
    
    ### Why are the changes needed?
    
    Improve the readability of logical plans.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Manual test:
    ```sql
    select id + 1 as a1, a1 + 2 as a3 from range(10);
    ```
    Before this PR:
    ```
    Project [(id#2L + 1) AS a1#0, (lateralAliasReference('a1, a1, 'a1) + 2) AS 
a3#1]
    +- Range (0, 10, step=1, splits=None)
    ```
    
    After this  PR:
    ```
    Project [(id#2L + 1) AS a1#0, (lateralAliasReference(a1) + 2) AS a3#1]
    +- Range (0, 10, step=1, splits=None)
    ```
    
    Closes #41359 from wangyum/SPARK-43859.
    
    Authored-by: Yuming Wang <yumw...@ebay.com>
    Signed-off-by: Yuming Wang <yumw...@ebay.com>
---
 .../org/apache/spark/sql/catalyst/expressions/namedExpressions.scala     | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
index 4f4fbc07b16..c0ab13c4e9e 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
@@ -459,6 +459,7 @@ case class LateralColumnAliasReference(ne: NamedExpression, 
nameParts: Seq[Strin
   override def dataType: DataType = ne.dataType
   override def prettyName: String = "lateralAliasReference"
   override def sql: String = s"$prettyName($name)"
+  override def toString: String = sql
 
   final override val nodePatterns: Seq[TreePattern] = 
Seq(LATERAL_COLUMN_ALIAS_REFERENCE)
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org