cloud-fan commented on a change in pull request #27866: [SPARK-31110][SQL]
refine sql doc for SELECT
URL: https://github.com/apache/spark/pull/27866#discussion_r390394463
##########
File path: docs/sql-ref-syntax-qry-select-orderby.md
##########
@@ -47,20 +47,18 @@ ORDER BY { expression [ sort_direction | nulls_sort_oder ]
[ , ... ] }
</dd>
<dt><code><em>nulls_sort_order</em></code></dt>
<dd>
- Optionally specifies whether NULL values are returned before/after
non-NULL values, based on the
- sort direction. In Spark, NULL values are considered to be lower than any
non-NULL values by default.
- Therefore the ordering of NULL values depend on the sort direction. If
<code>null_sort_order</code> is
- not specified, then NULLs sort first if sort order is <code>ASC</code> and
NULLS sort last if
- sort order is <code>DESC</code>.<br><br>
+ Optionally specifies whether NULL values are returned before/after
non-NULL values. If
+ <code>null_sort_order</code> is not specified, then NULLs sort first if
sort order is
+ <code>ASC</code> and NULLS sort last if sort order is
<code>DESC</code>.<br><br>
Review comment:
We don't need to mention `In Spark, NULL values are considered to be lower
than any non-NULL values`, as it's not related to the user-facing behavior.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
