ci: fix flaky test (#411)

zeroshade · web-flow · commit 2e570031ca8c · 2025-06-18T11:27:21.000-04:00
### Rationale for this change
Fixing flaky bloom filter memory check test

### What changes are included in this PR?
Added an additional runtime.GC() call to ensure the releases are called.
diff --git a/parquet/metadata/bloom_filter_reader_test.go b/parquet/metadata/bloom_filter_reader_test.go
@@ -52,6 +52,7 @@ func (suite *BloomFilterBuilderSuite) SetupTest() {
 
 func (suite *BloomFilterBuilderSuite) TearDownTest() {
 	runtime.GC() // we use setfinalizer to clean up the buffers, so run the GC
+	runtime.GC()
 	suite.mem.AssertSize(suite.T(), 0)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,7 @@ func (suite *BloomFilterBuilderSuite) SetupTest() {`
`52`	`52`
`53`	`53`	`func (suite *BloomFilterBuilderSuite) TearDownTest() {`
`54`	`54`	`runtime.GC() // we use setfinalizer to clean up the buffers, so run the GC`
	`55`	`+ runtime.GC()`
`55`	`56`	`suite.mem.AssertSize(suite.T(), 0)`
`56`	`57`	`}`
`57`	`58`