You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by sb...@apache.org on 2019/04/04 21:33:08 UTC

[arrow] branch master updated: ARROW-5119: [Go] fix Boolean stringer implementation

This is an automated email from the ASF dual-hosted git repository.

sbinet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 21b23f7  ARROW-5119: [Go] fix Boolean stringer implementation
21b23f7 is described below

commit 21b23f79730cb050e762820edb85a89aecd4e13c
Author: Sebastien Binet <bi...@cern.ch>
AuthorDate: Thu Apr 4 23:32:53 2019 +0200

    ARROW-5119: [Go] fix Boolean stringer implementation
    
    Author: Sebastien Binet <bi...@cern.ch>
    
    Closes #4112 from sbinet/issue-5119 and squashes the following commits:
    
    8eb4b191 <Sebastien Binet> ARROW-5119:  fix Boolean stringer implementation
---
 go/arrow/array/boolean.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/go/arrow/array/boolean.go b/go/arrow/array/boolean.go
index 68de951..b8bcf24 100644
--- a/go/arrow/array/boolean.go
+++ b/go/arrow/array/boolean.go
@@ -55,7 +55,7 @@ func (a *Boolean) Value(i int) bool {
 func (a *Boolean) String() string {
 	o := new(strings.Builder)
 	o.WriteString("[")
-	for i := range a.values {
+	for i := 0; i < a.Len(); i++ {
 		if i > 0 {
 			fmt.Fprintf(o, " ")
 		}