[GitHub] [arrow] AlenkaF edited a comment on pull request #12704: ARROW-15428: [Python] Address docstrings in Parquet classes and functions

[GitBox <gi...@apache.org>]

AlenkaF edited a comment on pull request #12704:
URL: https://github.com/apache/arrow/pull/12704#issuecomment-1080291719


   The reason for the failing test is added deprecation warning to `common_metadata` property of `ParquetDataset`:
   
   ```python
   In [1]: import pytest
   In [2]: import pyarrow.parquet as pq
   
   In [3]: with pytest.warns(None) as record:
       ...:     pq.read_table('v0.7.1.parquet',
       ...:                   use_legacy_dataset=True)
       ...: len(record)
   Out[3]: 2
   
   In [4]: record[0].message
   Out[4]: DeprecationWarning("Passing 'use_legacy_dataset=True' to get the legacy behaviour is deprecated as of pyarrow 8.0.0, and the legacy implementation will be removed in a future version.")
   
   In [5]: record[1].message
   Out[5]: DeprecationWarning("'ParquetDataset.common_metadata' attribute is deprecated as of pyarrow 5.0.0 and will be removed in a future version.")
   ```
   I am not sure why the warning for `common_metadata` property is saved and none of the other deprecated `ParquetDataset` properties are.
   
   I will corrected the test to include the new warning and will keep the change for `common_metadata`.


-- 
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-unsubscribe@arrow.apache.org

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