Skip to content

test(core): fix test failure with nd_array memory leak#6297

Merged
bluestreak01 merged 1 commit intomasterfrom
fix-fuzz-nd-array-mem-leak
Oct 22, 2025
Merged

test(core): fix test failure with nd_array memory leak#6297
bluestreak01 merged 1 commit intomasterfrom
fix-fuzz-nd-array-mem-leak

Conversation

@ideoma
Copy link
Copy Markdown
Collaborator

@ideoma ideoma commented Oct 22, 2025

Fixes:

2025-10-22T06:25:20.7617842Z java.lang.AssertionError: Memory usage by tag: NATIVE_ND_ARRAY, difference: 630 expected:<0> but was:<630>
2025-10-22T06:25:20.7618692Z 	at org.junit.Assert.fail(Assert.java:89)
2025-10-22T06:25:20.7619252Z 	at org.junit.Assert.failNotEquals(Assert.java:835)
2025-10-22T06:25:20.7619793Z 	at org.junit.Assert.assertEquals(Assert.java:647)
2025-10-22T06:25:20.7620358Z 	at io.questdb.test.tools.TestUtils$LeakCheck.close(TestUtils.java:2521)
2025-10-22T06:25:20.7620988Z 	at io.questdb.test.tools.TestUtils.assertMemoryLeak(TestUtils.java:778)
2025-10-22T06:25:20.7621610Z 	at io.questdb.test.cairo.o3.AbstractO3Test.assertMemoryLeak(AbstractO3Test.java:218)
2025-10-22T06:25:20.7622463Z 	at io.questdb.test.cairo.o3.AbstractO3Test.executeVanilla(AbstractO3Test.java:320)
2025-10-22T06:25:20.7623103Z 	at io.questdb.test.cairo.o3.AbstractO3Test.executeWithPool(AbstractO3Test.java:363)
2025-10-22T06:25:20.7623871Z 	at io.questdb.test.cairo.o3.AbstractO3Test.executeWithPool(AbstractO3Test.java:351)
2025-10-22T06:25:20.7628048Z 	at io.questdb.test.cairo.fuzz.O3MaxLagFuzzTest.testFuzzParallel(O3MaxLagFuzzTest.java:73)

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 22, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-fuzz-nd-array-mem-leak

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bluestreak01 bluestreak01 merged commit 812368b into master Oct 22, 2025
34 checks passed
@bluestreak01 bluestreak01 deleted the fix-fuzz-nd-array-mem-leak branch October 22, 2025 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bluestreak01 bluestreak01 bluestreak01 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ideoma @bluestreak01