Skip to content

GH-1037: Release ArrowBufs in VectorLoader on decompression failure#1146

Open
Kaustav-Sarkar wants to merge 1 commit into
apache:mainfrom
Kaustav-Sarkar:gh-1037
Open

GH-1037: Release ArrowBufs in VectorLoader on decompression failure#1146
Kaustav-Sarkar wants to merge 1 commit into
apache:mainfrom
Kaustav-Sarkar:gh-1037

Conversation

@Kaustav-Sarkar
Copy link
Copy Markdown
Contributor

Summary

  • Fix resource leak in VectorLoader when buffer decompression fails mid-loop
  • Wrap decompression and loadFieldBuffers in try-finally to ensure all already-decompressed ArrowBufs are released on failure

Test plan

  • Existing TestVectorLoader tests pass
  • Verified decompressed buffers are released when a subsequent decompression throws

@github-actions

This comment has been minimized.

@lidavidm lidavidm added the bug-fix PRs that fix a big. label May 13, 2026
@Kaustav-Sarkar Kaustav-Sarkar changed the title GH-1037: Release ArrowBufs in VectorLoader on decompression failure GH-1037 : Release ArrowBufs in VectorLoader on decompression failure May 14, 2026
@Kaustav-Sarkar Kaustav-Sarkar changed the title GH-1037 : Release ArrowBufs in VectorLoader on decompression failure GH-1037: Release ArrowBufs in VectorLoader on decompression failure May 14, 2026
@Kaustav-Sarkar
Copy link
Copy Markdown
Contributor Author

I do not understand the build error

Run actions/github-script@v9
{ issue: 1037 }
[]
Did not find link to https://github.com/apache/arrow-java/issues/1037
Error: Missing link to issue in title

Its literally redirecting to the issue link

Screenshot 2026-05-14 at 10 26 09 AM

@Kaustav-Sarkar Kaustav-Sarkar changed the title GH-1037: Release ArrowBufs in VectorLoader on decompression failure apacheGH-1037: Release ArrowBufs in VectorLoader on decompression failure May 14, 2026
@Kaustav-Sarkar Kaustav-Sarkar changed the title apacheGH-1037: Release ArrowBufs in VectorLoader on decompression failure GH-1037: Release ArrowBufs in VectorLoader on decompression failure May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix PRs that fix a big.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants