cloud-fan commented on code in PR #48940: URL: https://github.com/apache/spark/pull/48940#discussion_r1859862223
########## sql/core/src/test/resources/sql-tests/inputs/pipe-operators.sql: ########## @@ -241,6 +241,85 @@ table t table t |> extend *; +-- SET operators: positive tests. +--------------------------------- + +-- Setting with a constant. +-- The indicated column is not the last column in the table, and the SET operator will replace it +-- with the new value in its existing position. +table t +|> set x = 1; + +-- Setting with an attribute. +table t +|> set y = x; + +-- Setting with an expression. +table t +|> extend 1 as z +|> set z = x + length(y); + +-- Setting two times. +table t +|> extend 1 as z +|> extend 2 as zz +|> set z = x + length(y), zz = x + 1; + +table other +|> extend 3 as c +|> set a = b, b = c; + +-- Setting two times with a lateral reference. Review Comment: Furthermore, if we allow it, do we still need to check the uniqueness of the SET keys? People may want to do `SET c = c + 1, c = c * 2`, and this can be well supported because we use one `Project` per SET key. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
