raulcd commented on code in PR #48789:
URL: https://github.com/apache/arrow/pull/48789#discussion_r2675358510


##########
docs/source/developers/cpp/building.rst:
##########
@@ -314,6 +314,25 @@ The unit tests are not built by default. After building, 
one can also invoke
 the unit tests using the ``ctest`` tool provided by CMake (note that ``test``
 depends on ``python`` being available).
 
+.. note::
+   If you are building with tests (``-DARROW_BUILD_TESTS=ON``), you must ensure
+   the test data submodules are initialized and the environment variables
+   ``ARROW_TEST_DATA`` and ``PARQUET_TEST_DATA`` are set. Without these, 
several
+   tests (especially IPC and Parquet tests) will fail with an ``IOError``.
+
+   To initialize submodules, run:
+
+   .. code-block:: shell
+
+      $ git submodule update --init --recursive
+
+   Then set the variables to the absolute paths of your testing data folders:
+
+   .. code-block:: shell
+
+      $ export ARROW_TEST_DATA="<absolute_path_to_arrow>/testing/data"
+      $ export 
PARQUET_TEST_DATA="<absolute_path_to_arrow>/cpp/submodules/parquet-testing/data"
+  

Review Comment:
   There seems to be some whitespaces on this line which makes the linter fail, 
can you remove the whitespaces?
   ```
    Sphinx 
Lint..............................................................Failed
   - hook id: sphinx-lint
   - exit code: 1
   
   docs/source/developers/cpp/building.rst:335: trailing whitespace 
(trailing-whitespace)
   ```
   
   ```suggestion
   
   ```



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