You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/02/09 14:53:22 UTC

[GitHub] [spark] steven-aerts commented on pull request #35426: [SPARK-38130][SQL] Remove array_sort orderable entries check

steven-aerts commented on pull request #35426:
URL: https://github.com/apache/spark/pull/35426#issuecomment-1033844679


   Hi @cloud-fan,
   when `array_sort` does not have the comparator, it falls back to the `ArraySort.defaultComparator` which in itself is an expression containing a `LessThan` which does the order ability check, and generate a message with or without this patch.  See the [above comment](https://github.com/apache/spark/pull/35426#discussion_r801304946) for more details.


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org