[ARROW-5201] [Python] Import ABCs from collections is deprecated in Python 3.7

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.14.0
• Component/s: Python
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/21676
• Assignee: Joris Van den Bossche
• Reporter: Joris Van den Bossche
• Created: 23/Apr/19 09:15
• Updated: 11/Jan/23 07:38
• Resolved: 24/Apr/19 09:09

Description

From running the tests, I see a few deprecation warnings related to that on Python 3, abstract base classes should be imported from `collections.abc` instead of `collections`:

pyarrow/tests/test_array.py:808
  /home/joris/scipy/repos/arrow/python/pyarrow/tests/test_array.py:808: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    pa.struct([pa.field('a', pa.int64()), pa.field('b', pa.string())]))

pyarrow/tests/test_table.py:18
  /home/joris/scipy/repos/arrow/python/pyarrow/tests/test_table.py:18: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    from collections import OrderedDict, Iterable

pyarrow/tests/test_feather.py::TestFeatherReader::test_non_string_columns
  /home/joris/scipy/repos/arrow/python/pyarrow/pandas_compat.py:294: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    elif isinstance(name, collections.Sequence):

Those could be imported depending on python 2/3 in the ``pyarrow.compat`` module.

Issue Links

• duplicates: ARROW-4934 [Python] Address deprecation notice that will be a bug in Python 3.8
• links to: GitHub Pull Request #4187