milesgranger commented on code in PR #14096: URL: https://github.com/apache/arrow/pull/14096#discussion_r970595559
########## python/pyarrow/parquet/core.py: ########## @@ -16,23 +16,22 @@ # under the License. -from collections import defaultdict -from concurrent import futures -from contextlib import nullcontext -from functools import partial, reduce - -import sys -import json -from collections.abc import Collection -import numpy as np -import os -import re -import operator -import urllib.parse -import warnings - -import pyarrow as pa -import pyarrow.lib as lib +from collections import defaultdict as _defaultdict Review Comment: but I do agree, renaming isn't needed if we are just going to keep `__all__` defined in this module. -- 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]
