amoeba commented on code in PR #43629:
URL: https://github.com/apache/arrow/pull/43629#discussion_r1711973932
##########
docs/source/cpp/tutorials/compute_tutorial.rst:
##########
@@ -41,6 +41,9 @@ Before continuing, make sure you have:
1. An Arrow installation, which you can set up here: :doc:`/cpp/build_system`
+ - Be sure that your arrow installation
Review Comment:
```suggestion
- Be sure that your Arrow installation
```
##########
docs/source/cpp/tutorials/compute_tutorial.rst:
##########
@@ -49,8 +52,8 @@ Setup
Before running some computations, we need to fill in a couple gaps:
1. We need to include necessary headers.
-
-2. ``A main()`` is needed to glue things together.
+/cpp/building.html#optional-components
Review Comment:
```suggestion
```
Was this added unintentionally?
##########
docs/source/cpp/tutorials/compute_tutorial.rst:
##########
@@ -41,6 +41,9 @@ Before continuing, make sure you have:
1. An Arrow installation, which you can set up here: :doc:`/cpp/build_system`
Review Comment:
Maybe just something like this instead?
```suggestion
1. An Arrow installation, which you can set up here:
:doc:`/cpp/build_system`. If you're compiling Arrow yourself, be sure you
compile with the compute module enabled (i.e., ``-DARROW_COMPUTE=ON``), see
:doc:`Optional Components. <cpp_build_optional_components>` for more
information.
```
Then line 45 can be removed.
--
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]
