[
https://issues.apache.org/jira/browse/SPARK-31393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
jiaan.geng updated SPARK-31393:
-------------------------------
Description:
Spark SQL exists some function no elegant implementation alias.
For example: BitwiseCount override the sql method
{code:java}
override def sql: String = s"bit_count(${child.sql})"
{code}
I don't think it's elegant enough.
Because `Expression` gives the following definitions.
{code:java}
def sql: String = {
val childrenSQL = children.map(_.sql).mkString(", ")
s"$prettyName($childrenSQL)"
}
{code}
By this definition, BitwiseCount should override `prettyName` method.
was:
Spark SQL exists some function no elegant implementation alias.
For example: BitwiseCount override the sql method
override def sql: String = s"bit_count(${child.sql})"
I don't think it's elegant enough.
Because `Expression` gives the following definitions.
```
def sql: String = {
val childrenSQL = children.map(_.sql).mkString(", ")
s"$prettyName($childrenSQL)"
}
```
By this definition, BitwiseCount should override `prettyName` method.
> Show the correct alias in a more elegant way
> --------------------------------------------
>
> Key: SPARK-31393
> URL: https://issues.apache.org/jira/browse/SPARK-31393
> Project: Spark
> Issue Type: Improvement
> Components: SQL
> Affects Versions: 3.1.0
> Reporter: jiaan.geng
> Priority: Major
>
> Spark SQL exists some function no elegant implementation alias.
> For example: BitwiseCount override the sql method
> {code:java}
> override def sql: String = s"bit_count(${child.sql})"
> {code}
> I don't think it's elegant enough.
> Because `Expression` gives the following definitions.
> {code:java}
> def sql: String = {
> val childrenSQL = children.map(_.sql).mkString(", ")
> s"$prettyName($childrenSQL)"
> }
> {code}
> By this definition, BitwiseCount should override `prettyName` method.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
