izveigor commented on code in PR #7008: URL: https://github.com/apache/arrow-datafusion/pull/7008#discussion_r1267924137
########## datafusion/core/tests/sqllogictests/test_files/array.slt: ########## @@ -149,7 +149,7 @@ select column1, column2, column3 from arrays_values_without_nulls; ### Array function tests -## make_array +## make_array (aliases: `make_list`) Review Comment: @jayzhan211 I have identified several reasons: 1) The idea was taken from popular database: DuckDB. So, in my opinion, users of this database can more easily adapt to our array functions. (Documentation: https://duckdb.org/docs/sql/functions/nested). 2) `array` and `list` prefix or postfix can separate mutable or immutable objects. (See discussion: https://github.com/apache/arrow-datafusion/discussions/6855 (About aliases)). 3) Some people prefer to use one prefix/postfix over another. -- 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]
