FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GH-1037: Release ArrowBufs in VectorLoader on decompression failure by Kaustav-Sarkar · Pull Request #1146 · apache/arrow-java · GitHub

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

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

This comment has been minimized.

lidavidm added the bug-fix PRs that fix a big. label May 13, 2026
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 changed the title GH-1037 : Release ArrowBufs in VectorLoader on decompression failure GH-1037: Release ArrowBufs in VectorLoader on decompression failure May 14, 2026

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

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 changed the title apacheGH-1037: Release ArrowBufs in VectorLoader on decompression failure GH-1037: Release ArrowBufs in VectorLoader on decompression failure May 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL