HyukjinKwon commented on code in PR #48789:
URL: https://github.com/apache/arrow/pull/48789#discussion_r2674698943
##########
docs/source/developers/cpp/building.rst:
##########
@@ -314,14 +314,33 @@ 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"
+
On some Linux distributions, running the test suite might require setting an
explicit locale. If you see any locale-related errors, try setting the
environment variable (which requires the ``locales`` package or equivalent):
.. code-block::
$ export LC_ALL="en_US.UTF-8"
-
+
Review Comment:
```suggestion
```
no biggie but probably better to remove :-).
--
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]
