dtenedor commented on code in PR #48940: URL: https://github.com/apache/spark/pull/48940#discussion_r1861133636
########## 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: You're right, both of these are possible now. I removed the duplicate key assignment check and these cases work now. I added a couple test cases to cover it. -- 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]
