AlenkaF commented on code in PR #46591:
URL: https://github.com/apache/arrow/pull/46591#discussion_r2111299759


##########
docs/source/python/install.rst:
##########
@@ -98,6 +98,24 @@ a custom path to the database from Python:
    >>> import pyarrow as pa
    >>> pa.set_timezone_db_path("custom_path")
 
+You may encounter problems writing datetime data to an ORC file if you install
+pyarrow with pip. One possible solution to fix this problem:
+
+   1. Install tzdata with ``pip install tzdata``
+
+   2. Set the environmental variable ``TZDIR`` = 
``path\to\.venv\Lib\site-packages\tzdata\``
+
+You can find where ``tzdata`` is installed with the following python
+commands:
+
+.. code-block:: python
+
+   >>> import tzdata
+   >>> print(tzdata.__file__)
+   path\to\.venv\Lib\site-packages\tzdata\__init__.py
+
+You only need to set the ``TZDIR`` environmental variable to
+``path\to\.venv\Lib\site-packages\tzdata\`` in the above example.

Review Comment:
   Removing this part as the path and the environment variable are already 
mentioned in the numbered list.
   
   ```suggestion
   ```



##########
docs/source/python/install.rst:
##########
@@ -98,6 +98,24 @@ a custom path to the database from Python:
    >>> import pyarrow as pa
    >>> pa.set_timezone_db_path("custom_path")
 
+You may encounter problems writing datetime data to an ORC file if you install
+pyarrow with pip. One possible solution to fix this problem:
+
+   1. Install tzdata with ``pip install tzdata``
+

Review Comment:
   ```suggestion
   ```



##########
docs/source/python/install.rst:
##########
@@ -98,6 +98,24 @@ a custom path to the database from Python:
    >>> import pyarrow as pa
    >>> pa.set_timezone_db_path("custom_path")
 
+You may encounter problems writing datetime data to an ORC file if you install
+pyarrow with pip. One possible solution to fix this problem:
+
+   1. Install tzdata with ``pip install tzdata``
+
+   2. Set the environmental variable ``TZDIR`` = 
``path\to\.venv\Lib\site-packages\tzdata\``

Review Comment:
   ```suggestion
      2. Set the environment variable ``TZDIR`` = 
``path\to\.venv\Lib\site-packages\tzdata\``
   ```



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to