Skip to content

GH-48667: [Python] Remove unused imports from python/pyarrow/__init__.py #48640

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
AlenkaF merged 1 commit into from
Jan 7, 2026
Merged

GH-48667: [Python] Remove unused imports from python/pyarrow/__init__.py #48640

AlenkaF merged 1 commit into from
Jan 7, 2026

Conversation

@zhengruifeng
Copy link
Contributor

@zhengruifeng zhengruifeng commented Dec 24, 2025 •
edited by github-actions bot
Loading

What changes are included in this PR?

Remove unused imports from python/pyarrow/__init__.py

Are these changes tested?

Existing tests

Are there any user-facing changes?

No

@zhengruifeng zhengruifeng changed the title MINOR: [PYTHON] Remove unused imports from python/pyarrow/__init__.py MINOR: [PYTHON] Remove unused imports from python/pyarrow/__init__.py (GH#48667) Dec 26, 2025
@zhengruifeng zhengruifeng changed the title MINOR: [PYTHON] Remove unused imports from python/pyarrow/__init__.py (GH#48667) GH-48667: [PYTHON] Remove unused imports from python/pyarrow/__init__.py Dec 26, 2025
@github-actions
Copy link

github-actions bot commented Dec 26, 2025

⚠️ GitHub issue #48667 has been automatically assigned in GitHub to PR creator.

@AlenkaF AlenkaF changed the title GH-48667: [PYTHON] Remove unused imports from python/pyarrow/__init__.py GH-48667: [Python] Remove unused imports from python/pyarrow/__init__.py Jan 5, 2026
AlenkaF
AlenkaF approved these changes Jan 5, 2026
View reviewed changes
Copy link
Member

@AlenkaF AlenkaF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for cleaning this up!
@rok for a sanity check.

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Jan 5, 2026
raulcd
raulcd approved these changes Jan 7, 2026
View reviewed changes
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, those seem to have been added as workarounds for old issues which have been fixed now, the commits that added those imports:
9123dad
25d131b

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jan 7, 2026
@AlenkaF AlenkaF merged commit 2ed8665 into apache:main Jan 7, 2026
19 checks passed
@AlenkaF AlenkaF removed the awaiting merge Awaiting merge label Jan 7, 2026
@AlenkaF AlenkaF mentioned this pull request Jan 7, 2026
Closed
@zhengruifeng zhengruifeng deleted the init_unused_imports branch January 7, 2026 09:42
@zhengruifeng
Copy link
Contributor Author

zhengruifeng commented Jan 7, 2026

Thank you so much @raulcd @AlenkaF

@conbench-apache-arrow
Copy link

conbench-apache-arrow bot commented Jan 7, 2026

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 2ed8665.

There weren't enough matching historic benchmark results to make a call on whether there were regressions.

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raulcd raulcd raulcd approved these changes

@AlenkaF AlenkaF AlenkaF approved these changes

@rok rok Awaiting requested review from rok rok is a code owner

Assignees

No one assigned

Labels

Component: Python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zhengruifeng @raulcd @AlenkaF