HyukjinKwon commented on code in PR #43928:
URL: https://github.com/apache/spark/pull/43928#discussion_r1400213112


##########
python/pyspark/sql/udtf.py:
##########
@@ -82,13 +87,21 @@ class OrderingColumn:
     """
     Represents a single ordering column name for purposes of returning 
metadata from the 'analyze'
     method.
+
+    Parameters
+    ----------
+    name : str
+        The name of the partitioning column.
+    ascending : bool, default True
+        If this column is in an ascending order or not.
+    overrideNullsFirst : str, optional
+        If this is None, use the default behavior to sort NULL values first 
when sorting in ascending
+        order, or last when sorting in descending order. Otherwise, if this is 
True or False, override
+        the default behavior accordingly.

Review Comment:
   ```suggestion
           If this is None, use the default behavior to sort NULL values first 
when sorting in
           ascending order, or last when sorting in descending order. 
Otherwise, if this is
           True or False, override the default behavior accordingly.
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to