zeroshade commented on code in PR #14223:
URL: https://github.com/apache/arrow/pull/14223#discussion_r1093560359
##########
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:
I hadn't planned on doing it in this PR yet, there's some discussion that
should probably happen on whether or not a JSON representation of
run-end-encoded values should be expanded out (like dictionaries) or if there
should be an explicit run-end-encoded JSON format (would require some
specialization for nested cases since they wouldn't be able to just use `.Len`
on the array)
--
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]
