Matt Topol
@pitrou yea, i'm adding IPC roundtrip tests today. well, i'm adding sample run end encoded records to `go/arrow/internal/arrdata/arrdata.go` so all the tests which use the sample records in arrdata (including...
@pitrou IPC round trips have been added (the tests in `ipc/file_test.go` and `ipc/stream_test.go` both loop over the list of test records in `arrdata` and test a round trip IPC in...
I'll merge this EOD if no one has any objections.
I'll merge this at EOD if no one else has comments
I'll merge this at EOD if no one has any objections/comments
If there's no further comments before EOD today i'll merge this.
Would it be possible to get some more reviews here? I've been attempting to keep this up to date but I haven't dug too much further into building this until...
@vyasr This should be good for you to take another pass now. * I've updated the cmake to use `FetchContent` to pull nanoarrow at build time * I've shifted declarations...
@vyasr I think i've addressed all of your comments (or replied to them) let me know what you think and if I should mark this as `Ready for review` :smile:
> However, the conversion to host is a special case because what we typically want is for the input cudf objects to remain valid after the host conversion is completed....