lidavidm commented on a change in pull request #11129:
URL: https://github.com/apache/arrow/pull/11129#discussion_r706476234
##########
File path: docs/source/cpp/compute.rst
##########
@@ -542,16 +542,17 @@ cast to the :ref:`common numeric type
<common-numeric-type>` before comparison),
or two inputs of Binary- or String-like types, or two inputs of Temporal types.
If any input is dictionary encoded it will be expanded for the purposes of
comparison. If any of the input elements in a pair is null, the corresponding
-output element is null.
-
-+--------------------------+------------+---------------------------------------------+---------------------+
-| Function names | Arity | Input types
| Output type |
-+==========================+============+=============================================+=====================+
-| equal, not_equal | Binary | Numeric, Temporal, Binary- and
String-like | Boolean |
-+--------------------------+------------+---------------------------------------------+---------------------+
-| greater, greater_equal, | Binary | Numeric, Temporal, Binary- and
String-like | Boolean |
-| less, less_equal | |
| |
-+--------------------------+------------+---------------------------------------------+---------------------+
+output element is null. Decimal arguments will be promoted in the same way as
+for ``add`` and ``subtract``.
+
++--------------------------+------------+------------------------------------------------------+---------------------+
+| Function names | Arity | Input types
| Output type |
++==========================+============+======================================================+=====================+
+| equal, not_equal | Binary | Numeric, Temporal, Decimal, Binary-
and String-like | Boolean |
Review comment:
Fair, "Numeric" is defined up top as often including decimal. The blurb
about promotion here should suffice, I think; I also added a blurb about
decimal promotion to the arithmetic section.
--
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]
