Skip to content

Commit

Permalink
query/physicalplan: Sampler requires LessOrEqual 1024 bytes allocations
Browse files Browse the repository at this point in the history
  • Loading branch information
metalmatze committed Nov 25, 2024
1 parent cc92765 commit dd0ae83
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion query/physicalplan/sampler_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ func Test_Sampler_MaxSizeAllocation(t *testing.T) {
allocator: memory.NewCheckedAllocator(memory.NewGoAllocator()),
}
t.Cleanup(func() {
require.Equal(t, allocator.maxUsed, 1024) // Expect the most we allocated was 1024 bytes during materialization
require.LessOrEqual(t, allocator.maxUsed, 1024) // Expect the most we allocated was 1024 bytes during materialization
allocator.allocator.AssertSize(t, 0)
})
s := NewReservoirSampler(10, 200, allocator)
Expand Down

0 comments on commit dd0ae83

Please sign in to comment.