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_r390394681
##########
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>
<ol>
- <li> If <code>NULLS FIRST</code> (the default) is specified, then NULL
values are returned first
+ <li> If <code>NULLS FIRST</code> is specified, then NULL values are
returned first
Review comment:
the default value depends on the sort order.
----------------------------------------------------------------
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]
