zeroshade commented on code in PR #14223:
URL: https://github.com/apache/arrow/pull/14223#discussion_r1093689145
##########
go/arrow/array/util_test.go:
##########
@@ -430,6 +430,10 @@ func TestArrRecordsJSONRoundTrip(t *testing.T) {
// float weirdness due to their size, so smaller tests
will work fine.
continue
}
+ if k == "run_end_encoded" {
+ // unmarshalling json to run_end_encoded not yet
implemented
Review Comment:
after chatting with @felipecrv and coming to a decision on whether or not we
wanted to expand the array (like how JSON marshalling of dictionary arrays
expands the array), I've implemented JSON unmarshalling and added it to this
PR, removing this conditional check.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
