jorisvandenbossche commented on code in PR #14096: URL: https://github.com/apache/arrow/pull/14096#discussion_r976513774
########## 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: Given that this is for importing all symbols in a submodule (core.py) to the main public module (pyarrow.parquet), I think the `__all__` approach is fine, and we can keep doing that. (so I would revert the other renamings) -- 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]
