jayzhan211 commented on code in PR #6796: URL: https://github.com/apache/arrow-datafusion/pull/6796#discussion_r1278474676
########## datafusion/core/tests/sqllogictests/test_files/array.slt: ########## @@ -1794,9 +1802,150 @@ select make_array(f0) from fixed_size_list_array ---- [[1, 2], [3, 4]] +## Unnest -### Delete tables +# Set target partitions to 1 for deterministic results +statement ok +set datafusion.execution.target_partitions = 1; + +query ?? +select unnest(make_array(1,2,3)), + unnest(make_array(4,5)) +; +---- +1 4 +2 5 +3 NULL + +query ??? +select unnest(make_array(1,2,3)), + unnest(make_array(4,5)), + unnest(make_array(6,7,8,9)) +; +---- +1 4 6 +2 5 7 +3 NULL 8 +NULL NULL 9 + +query ??? +select unnest(make_array(1,2,3,4,5)), + unnest(make_array(6,7)), + unnest(make_array(8,9,10,11,22,33)) +; +---- +1 6 8 +2 7 9 +3 NULL 10 +4 NULL 11 +5 NULL 22 +NULL NULL 33 + +# Select From + +query IIIII +select * from unnest( + make_array(1), + make_array(2,3), + make_array(4,5,6), + make_array(7,8), + make_array(9) +); +---- Review Comment: I also agree this syntax is better if we like partial unnest, e.g. select unnest(a), b, unnest(c) from table. Clickhouse has the similar syntax https://clickhouse.com/docs/en/sql-reference/functions/array-join ########## datafusion/core/tests/sqllogictests/test_files/array.slt: ########## @@ -1794,9 +1802,150 @@ select make_array(f0) from fixed_size_list_array ---- [[1, 2], [3, 4]] +## Unnest -### Delete tables +# Set target partitions to 1 for deterministic results +statement ok +set datafusion.execution.target_partitions = 1; + +query ?? +select unnest(make_array(1,2,3)), + unnest(make_array(4,5)) +; +---- +1 4 +2 5 +3 NULL + +query ??? +select unnest(make_array(1,2,3)), + unnest(make_array(4,5)), + unnest(make_array(6,7,8,9)) +; +---- +1 4 6 +2 5 7 +3 NULL 8 +NULL NULL 9 + +query ??? +select unnest(make_array(1,2,3,4,5)), + unnest(make_array(6,7)), + unnest(make_array(8,9,10,11,22,33)) +; +---- +1 6 8 +2 7 9 +3 NULL 10 +4 NULL 11 +5 NULL 22 +NULL NULL 33 + +# Select From + +query IIIII +select * from unnest( + make_array(1), + make_array(2,3), + make_array(4,5,6), + make_array(7,8), + make_array(9) +); +---- Review Comment: I agree this syntax is better if we like partial unnest, e.g. select unnest(a), b, unnest(c) from table. Clickhouse has the similar syntax https://clickhouse.com/docs/en/sql-reference/functions/array-join -- 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]
