Skip to content

Commit

Permalink
MINOR: Fix build double declaration (apache#36876)
Browse files Browse the repository at this point in the history
### Rationale for this change
apache#36811 and apache#36854 both introduced a helper in their tests, but the result after merge was a collision causing it to be declared twice which made the Go build fail. By removing one of the duplicate declarations the build is fixed.

Authored-by: Matt Topol <[email protected]>
Signed-off-by: Matt Topol <[email protected]>
  • Loading branch information
zeroshade authored Jul 25, 2023
1 parent adfd482 commit 5a9240f
Showing 1 changed file with 0 additions and 20 deletions.
20 changes: 0 additions & 20 deletions go/arrow/array/concat_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -744,26 +744,6 @@ func TestConcatOverflowRunEndEncoding(t *testing.T) {
}
}

type panicAllocator struct {
n int
memory.Allocator
}

func (p *panicAllocator) Allocate(size int) []byte {
if size > p.n {
panic("panic allocator")
}
return p.Allocator.Allocate(size)
}

func (p *panicAllocator) Reallocate(size int, b []byte) []byte {
return p.Allocator.Reallocate(size, b)
}

func (p *panicAllocator) Free(b []byte) {
p.Allocator.Free(b)
}

func TestConcatPanic(t *testing.T) {
mem := memory.NewCheckedAllocator(memory.DefaultAllocator)
defer mem.AssertSize(t, 0)
Expand Down

0 comments on commit 5a9240f

Please sign in to comment.