Skip to content

Release/0.9.1#187

Merged
Ian Streeter (istreeter) merged 2 commits intomasterfrom
release/0.9.1
Feb 18, 2026
Merged

Release/0.9.1#187
Ian Streeter (istreeter) merged 2 commits intomasterfrom
release/0.9.1

Conversation

@istreeter
Copy link
Contributor

No description provided.

Open database files via FileInputStream and pass to the InputStream-based
DatabaseReader.Builder constructor, instead of passing a File which
internally uses FileChannel.read into a heap ByteBuffer. The FileChannel
path causes the JDK to allocate and cache temporary direct ByteBuffers
in thread-local storage (sun.nio.ch.Util.BufferCache), leading to
off-heap memory growth in scenarios where the database is initialized
repeatedly.
Copy link
Member

Choose a reason for hiding this comment

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

2012-2022 should be updated in README as well.

@coveralls
Copy link

Coveralls (coveralls) commented Feb 18, 2026

Pull Request Test Coverage Report for Build 22138085634

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 85.401%

Totals Coverage Status
Change from base Build 21751687977: 0.3%
Covered Lines: 117
Relevant Lines: 137

💛 - Coveralls

Copy link
Member

Choose a reason for hiding this comment

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

thanks!

@istreeter Ian Streeter (istreeter) merged commit 5186a1c into master Feb 18, 2026
1 check passed
@istreeter Ian Streeter (istreeter) deleted the release/0.9.1 branch February 18, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants