Tom-Newton commented on code in PR #36627: URL: https://github.com/apache/arrow/pull/36627#discussion_r1260849702
########## python/pyarrow/tests/test_dataset.py: ########## @@ -16,31 +16,29 @@ # under the License. import contextlib -import os -import posixpath import datetime +import os import pathlib import pickle +import posixpath import sys -import textwrap import tempfile +import textwrap import threading import time - from urllib.parse import quote import numpy as np -import pytest - import pyarrow as pa import pyarrow.compute as pc import pyarrow.csv -import pyarrow.json import pyarrow.feather import pyarrow.fs as fs -from pyarrow.tests.util import (change_cwd, _filesystem_uri, - FSProtocolClass, ProxyHandler, - _configure_s3_limited_user) +import pyarrow.json +import pytest +from pyarrow.tests.util import (FSProtocolClass, ProxyHandler, + _configure_s3_limited_user, _filesystem_uri, + change_cwd) Review Comment: I ran `archery lint --python --fix` which appears to have shuffled around a few bits in this file that I didn't change. Presumably this is a good thing. -- 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]
