Github user DonnyZone commented on the issue:

    https://github.com/apache/spark/pull/19559
  
    @gatorsmile @gatorsmile 
    There are still two issues need to be figured out.
    (1)It will be complicated to determine whether a literal function should be 
resolved as Expression or NamedExpression.
    Current fix just resolves them as NamedExpressions (i.e., Alias). 
    However, this leads to different schema in some cases, for example, the 
end-to-end test sql.
    `select current_date = current_date()`
    The output schema will be
    `struct<(current_date() AS ’current_date()‘ = current_date()):boolean>`
    (2)Shall we also support the feature in ResolveMissingReference rule?
    e.g., `select id from table order by current_date`
    The same logic in different rules brings redundant code.


---

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

Reply via email to