Minor fix to SPARK-6958: Improve Python docstring for DataFrame.sort.

[rxin]

As a follow up PR to #5544.

cc @davies

[davies]

In Python, it's called bool

[rxin]

I think bool is the name of the function, but the data type is still boolean?

see

http://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.sort.html#pandas.DataFrame.sort

https://docs.python.org/2/library/stdtypes.html#boolean-values

https://docs.python.org/2/c-api/bool.html

[davies]

The type for boolean is bool:

>>> type(True)
<type 'bool'>
>>> type(1)
<type 'int'>

Similar to int for integer.

[davies]

It's weird to see we put boolean and int together, it should be either boolean/integer or bool/int.

[rxin]

Alright I removed int, since 1, 0 has the same value as True, False anyway.

[davies]

Could you also change the tests in doc string to use True and False?

[rxin]

then that's different from the Pandas examples?

[davies]

yeah, no perfect.

[SparkQA]

Test build #30488 has finished for PR 5558 at commit d2dcf24.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[SparkQA]

Test build #30498 has finished for PR 5558 at commit f4c276f.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[davies]

LGTM.