Skip to content
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

[HUDI-8797] Fix the potential filesystem connection leak #12546

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[HUDI-8797] Fix the potential filesystem connection leak #12546

wants to merge 1 commit into from

Conversation

Zouxxyy
Copy link
Contributor

@Zouxxyy Zouxxyy commented Dec 27, 2024 •
edited
Loading

Change Logs

  • add filesystem connection leak check in HoodieSparkSqlTestBase
  • fix all potential filesystem connection leak

Impact

Risk level (write none, low medium or high below)

low

Documentation Update

none

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Dec 27, 2024
@Zouxxyy Zouxxyy changed the title [HUDI-8797] Fix Filesystem leak in ParquetFileFormat [HUDI-8797] Fix filesystem connection leak in ParquetFileFormat Dec 27, 2024
@Zouxxyy Zouxxyy marked this pull request as draft December 27, 2024 12:43
@github-actions github-actions bot added size:M PR with lines of changes in (100, 300] and removed size:S PR with lines of changes in (10, 100] labels Dec 27, 2024
@Zouxxyy Zouxxyy changed the title [HUDI-8797] Fix filesystem connection leak in ParquetFileFormat [HUDI-8797] Fix the potential filesystem connection leak Dec 29, 2024
@Zouxxyy Zouxxyy force-pushed the dev/fs-open branch 3 times, most recently from 290aad6 to d59cdaa Compare December 29, 2024 16:37
@hudi-bot
Copy link

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@Zouxxyy Zouxxyy closed this Dec 30, 2024
@Zouxxyy Zouxxyy reopened this Dec 30, 2024
@danny0405
Copy link
Contributor

@Zouxxyy Is this PR ready for review now?

@Zouxxyy
Copy link
Contributor Author

Zouxxyy commented Jan 2, 2025

@Zouxxyy Is this PR ready for review now?

Not yet, encountered an unstable test case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size:M PR with lines of changes in (100, 300]
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Zouxxyy @hudi-bot @danny0405