Gschiavon commented on issue #25728: [SPARK-29020][SQL] Improving array_sort 
behaviour
URL: https://github.com/apache/spark/pull/25728#issuecomment-529994076
 
 
   > I see, is it that the ordering of nulls is different?
   > I think we'd at least want the doc of each to comment on the difference 
with the other very similar method, to make this clear. I suppose one day we 
may add control over null ordering and deprecate one in favor of the other.
   
   Okay @srowen , I'll explain the ordering of nulls for all as they are now:
   `sort_array` in `ascending order` nulls are at the `beginning`
   `sort_array` in `descending order` nulls are at the `end`
   `array_sort` in `ascending order` nulls are at the `end`
   
   And after the PR:
   `sort_array` will remain the same
   `array_sort` in `ascending order` nulls are at the `end`
   `array_sort` in `descending order` nulls are at the `end`
   
   So this way the actual behaviour of `array_sort` remains the same.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to