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 2026/07/04 17:03:25 UTC

(arrow-go) branch main updated: fix(array): clarify StringViewBuilder UnmarshalJSON error message (#894)

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-go.git


The following commit(s) were added to refs/heads/main by this push:
     new 7d4af07b fix(array): clarify StringViewBuilder UnmarshalJSON error message (#894)
7d4af07b is described below

commit 7d4af07b49fc7dc394b0bcf0910eed2fa1488400
Author: Minh Vu <vu...@gmail.com>
AuthorDate: Sat Jul 4 19:03:19 2026 +0200

    fix(array): clarify StringViewBuilder UnmarshalJSON error message (#894)
    
    The StringViewBuilder JSON decode error said "binary view builder"; report "string view builder" instead.
---
 arrow/array/string.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arrow/array/string.go b/arrow/array/string.go
index 00748576..559a216c 100644
--- a/arrow/array/string.go
+++ b/arrow/array/string.go
@@ -803,7 +803,7 @@ func (b *StringViewBuilder) UnmarshalJSON(data []byte) error {
 	}
 
 	if delim, ok := t.(json.Delim); !ok || delim != '[' {
-		return fmt.Errorf("binary view builder must unpack from json array, found %s", delim)
+		return fmt.Errorf("string view builder must unpack from json array, found %s", delim)
 	}
 
 	return b.Unmarshal(dec)