distutils is deprecated in Python 3.10

[tirkarthi]

https://www.python.org/dev/peps/pep-0632/#migration-advice

filesystem_spec/fsspec/implementations/tests/test_local.py

Line 12 in 0417b34

from distutils.version import LooseVersion

filesystem_spec/fsspec/spec.py

Line 7 in bd932c3

from distutils.version import LooseVersion

[isidentical]

It seems like we use it here:

filesystem_spec/fsspec/spec.py

Lines 91 to 101 in bd932c3

	pa_version = get_package_version_without_import("pyarrow")
	if pa_version and LooseVersion(pa_version) < LooseVersion("2.0"):
	try:
	import pyarrow as pa
	up = pa.filesystem.DaskFileSystem
	except ImportError: # pragma: no cover
	# pyarrow exists but doesn't import for some reason
	up = object
	else: # pragma: no cover
	up = object

I don't know whether that part of the code is still needed though, it seems like a temporary hack. CC: @martindurant

[martindurant]

Pyarrow < 2 ( https://pypi.org/project/pyarrow/#history ) would be about a year and a half old, should we just drop it, then? I vote yes.

[isidentical]

👍🏻

[isidentical]

We still use it in the tests @martindurant.

[isidentical]

filesystem_spec/fsspec/implementations/tests/test_local.py

Lines 349 to 362 in 0417b34

	def test_get_pyarrow_filesystem():
	pa = pytest.importorskip("pyarrow")
	fs = LocalFileSystem()
	if LooseVersion(pa.__version__) < LooseVersion("2.0"):
	assert isinstance(fs, pa.filesystem.FileSystem)
	assert fs._get_pyarrow_filesystem() is fs
	else:
	assert not isinstance(fs, pa.filesystem.FileSystem)
	class UnknownFileSystem(object):
	pass
	assert not isinstance(UnknownFileSystem(), pa.filesystem.FileSystem)

[martindurant]

I'm not too worried about deprecation in a test, but I can fix that too. This test became obsolete anyway.