Skip to content

Conversation

JaroslavTulach
Copy link
Member

@JaroslavTulach JaroslavTulach commented Sep 2, 2025

Pull Request Description

  • To effectively exchange huge chunks of memory among the "dual JVMs" ...
  • ... as needed by In-Memory table implementations, let's rely on direct byte buffers
  • with this PR we have a way for the two "dual JVMs" to exchange and share the same chunk of memory

Important Notes

  • care must be taken to work properly with GC & free
  • holding just the ByteBuffer instance doesn't prevent other JVM to GC and release its memory region
  • holding a pointer to the original Value object while working with the ByteBuffer should be enough to prevent GC

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • All code follows the
    Scala,
    Java,
  • Unit tests have been written where possible.

@JaroslavTulach JaroslavTulach added this to the 2025.3 Release milestone Sep 2, 2025
@JaroslavTulach JaroslavTulach self-assigned this Sep 2, 2025
@JaroslavTulach JaroslavTulach added the CI: No changelog needed Do not require a changelog entry for this PR. label Sep 2, 2025
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🌟 Q/A review in Issues Board Sep 3, 2025
@JaroslavTulach JaroslavTulach added the CI: Ready to merge This PR is eligible for automatic merge label Sep 3, 2025
@mergify mergify bot merged commit 8b8044e into develop Sep 3, 2025
76 checks passed
@github-project-automation github-project-automation bot moved this from 🌟 Q/A review to 🟢 Accepted in Issues Board Sep 3, 2025
@mergify mergify bot deleted the wip/jtulach/OtherByteBuffer branch September 3, 2025 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

-compiler CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge

Projects

Status: 🟢 Accepted

Development

Successfully merging this pull request may close these issues.

3 participants