jayzhan211 commented on code in PR #8121: URL: https://github.com/apache/arrow-datafusion/pull/8121#discussion_r1402815790
########## datafusion/sqllogictest/test_files/array.slt: ########## @@ -2427,6 +2469,18 @@ NULL 10 NULL 10 NULL 10 +query II +select array_length(arrow_cast(array[array[1, 2], array[3, 4]], 'LargeList(List(Int64))'), column3), array_length(arrow_cast(column1, 'LargeList(Int64)'), 1) from arrays_values; Review Comment: Mixing non-table with table test cases is quite confusing, not sure which line does the second one start from, can we separate them? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
