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)