jorisvandenbossche commented on code in PR #14096: URL: https://github.com/apache/arrow/pull/14096#discussion_r978368357
########## 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: I am fine with deprecating it, but I think we should initially do that with a DeprecationWarning instead of FutureWarning (those are less visible, and since this is a function that is used by other packages, not by end-users, ideally first those packages can solve the warning without their users seeing it). -- 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]
