Skip to content

Conversation

@JinwooHwang
Copy link
Contributor

Problem

geode-lucene:integrationTest fails with OutOfMemoryError: Java heap space during classpath scanning in CI environments.

Root Cause

Jakarta EE migration introduced ByteBuffersDirectory with different memory characteristics. The existing 12GB heap allocation is insufficient for the full integration test suite in resource-constrained CI environments.

Solution

  • Increased integrationTest.maxHeapSize from 12g to 16g in geode-lucene/build.gradle
  • No production code changes

Testing

  • Tests pass locally
  • Monitoring CI for successful completion

Impact

Low risk - Only affects test execution environment, no runtime changes.


Files Changed:

  • geode-lucene/build.gradle

For all changes, please confirm:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
  • Has your PR been rebased against the latest commit within the target branch (typically develop)?
  • Is your initial contribution a single, squashed commit?
  • Does gradlew build run cleanly?
  • Have you written or updated unit tests to verify your changes?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Problem:
geode-lucene:integrationTest fails with OutOfMemoryError: Java heap space
during classpath scanning (ClassGraph loading Gfsh commands) on CI environments.

Root Cause:
Jakarta EE migration introduced ByteBuffersDirectory which may have different
memory characteristics than the previous implementation. The existing 12GB heap
allocation is insufficient when running the full integration test suite in
resource-constrained CI environments.

Solution:
Increased integrationTest.maxHeapSize from 12g to 16g in geode-lucene/build.gradle.

Impact:
Low risk - only affects test execution memory allocation, no production code changes.
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.

1 participant