fix tests that are failing on github ci #36
Merged
Conversation
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
ekozilforce
approved these changes
Jun 9, 2025
dhagberg-sf
approved these changes
Jun 9, 2025
|
@dhagberg-sf I think we are okay to merge despite not updating the changelog here. Prob not worth tracking this on a changelog as nothing here impacts customers and its still ea. The changelog check was a pre-existing feature and we may need to refine it post-ga along with our rollout approach. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR addresses the Java Heap memory issue that was causing certain environments, macos and later ubuntu to fail github ci/cd test. Specifically the large data ingest tests would fail because the size of the payload was exceeding the default memory size limitation. Along with an increase in default heap memory allocation to allow the largest bulk job to be successful we are clearing the wiremock results between each bulk test.