You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ze...@apache.org on 2023/06/02 19:52:43 UTC
[arrow] branch main updated: MINOR: [Go] Use `arrow.ListLikeType` (#35893)
This is an automated email from the ASF dual-hosted git repository.
zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 5a55fb4f66 MINOR: [Go] Use `arrow.ListLikeType` (#35893)
5a55fb4f66 is described below
commit 5a55fb4f66cbd316568085129054b5ff1d2eb089
Author: Alex Shcherbakov <ca...@users.noreply.github.com>
AuthorDate: Fri Jun 2 22:52:30 2023 +0300
MINOR: [Go] Use `arrow.ListLikeType` (#35893)
### Rationale for this change
Follow-up for #35885
### What changes are included in this PR?
Removed `listLikeType` interface from `ipc.go` & swapped it for `arrow.ListLikeType`.
### Are these changes tested?
It's only a replace of the interface name.
### Are there any user-facing changes?
No.
Authored-by: candiduslynx <ca...@gmail.com>
Signed-off-by: Matt Topol <zo...@gmail.com>
---
go/arrow/ipc/file_reader.go | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/go/arrow/ipc/file_reader.go b/go/arrow/ipc/file_reader.go
index 658d79d330..a561352960 100644
--- a/go/arrow/ipc/file_reader.go
+++ b/go/arrow/ipc/file_reader.go
@@ -595,12 +595,7 @@ func (ctx *arrayLoaderContext) loadMap(dt *arrow.MapType) arrow.ArrayData {
return array.NewData(dt, int(field.Length()), buffers, []arrow.ArrayData{sub}, int(field.NullCount()), 0)
}
-type listLike interface {
- arrow.DataType
- Elem() arrow.DataType
-}
-
-func (ctx *arrayLoaderContext) loadList(dt listLike) arrow.ArrayData {
+func (ctx *arrayLoaderContext) loadList(dt arrow.ListLikeType) arrow.ArrayData {
field, buffers := ctx.loadCommon(dt.ID(), 2)
buffers = append(buffers, ctx.buffer())
defer releaseBuffers(buffers)