Skip to content

[MINOR] Fixing FS InputStream leaks#7741

Merged
alexeykudinkin merged 1 commit intoapache:masterfrom
onehouseinc:ak/strm-leak-fix
Jan 24, 2023
Merged

[MINOR] Fixing FS InputStream leaks#7741
alexeykudinkin merged 1 commit intoapache:masterfrom
onehouseinc:ak/strm-leak-fix

Conversation

@alexeykudinkin
Copy link
Contributor

Change Logs

Fixing FS FSDataInputStream leaks

Impact

Makes sure that allocated resources are cleaned up appropriately

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

Low

Documentation Update

N/A

Contributor's checklist

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

Copy link
Contributor

@danny0405 danny0405 left a comment

Choose a reason for hiding this comment

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

+1

@danny0405 danny0405 self-assigned this Jan 24, 2023
@danny0405 danny0405 added priority:medium Moderate impact; usability gaps writer-core labels Jan 24, 2023
@hudi-bot
Copy link
Collaborator

CI report:

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

@alexeykudinkin alexeykudinkin merged commit fc1831b into apache:master Jan 24, 2023
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Jan 31, 2023
Fixing `FSDataInputStream` leaks
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Mar 22, 2023
Fixing `FSDataInputStream` leaks
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Apr 5, 2023
Fixing `FSDataInputStream` leaks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:medium Moderate impact; usability gaps

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants