pitrou commented on a change in pull request #11883:
URL: https://github.com/apache/arrow/pull/11883#discussion_r765040729



##########
File path: docs/source/cpp/compute.rst
##########
@@ -911,6 +913,8 @@ String transforms
   If the input is not valid UTF8, then the output is undefined (but the size 
of output
   buffers will be preserved).
 
+* \(11) Performs a byte-level reverse. Does not make any assumptions about 
encoding.

Review comment:
       ```suggestion
   * \(11) Perform a byte-level reverse.
   ```

##########
File path: docs/source/cpp/compute.rst
##########
@@ -851,6 +851,8 @@ String transforms
 
+-------------------------+--------+-----------------------------------------+------------------------+-----------------------------------+-------+
 | binary_replace_slice    | Unary  | String-like                             | 
Binary- or String-like | :struct:`ReplaceSliceOptions`     | \(5)  |
 
+-------------------------+--------+-----------------------------------------+------------------------+-----------------------------------+-------+
+| binary_reverse          | Unary  | Binary                                  | 
Binary                 |                                   | \(11) |

Review comment:
       Hmm... can you renumber the notes so that they appear in ascending order 
in this table? It's a bit cumbersome, but doable :-)




-- 
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]


Reply via email to