Asura7969 commented on PR #8279: URL: https://github.com/apache/arrow-datafusion/pull/8279#issuecomment-1820359745
> > Since DuckDB has `array_sort` already, and I think there is no good reason that we should not follow theirs. Can you have the similar style like theirs? > > https://duckdb.org/docs/sql/functions/nested#sorting-lists > > Especially, I think array_sort(list, 'asc') or array_sort(list, 'desc') is more much straightforward then `true`, `false`. We need to check one of the result to know whether `true` is `asc` or `desc`. So as `null_first` > > I'll try I improved -- 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
