ARROW-3761: [R] Bindings for CompressedInputStream, CompressedOutputStream

[romainfrancois]

No description provided.

[codecov-io]

Codecov Report

Merging #2962 into master will increase coverage by 15.14%.
The diff coverage is n/a.

@@ Coverage Diff @@## master #2962 +/- ##
===========================================
+ Coverage 74.31% 89.45% +15.14% 
===========================================
Files 68 12 -56 Lines 5423 1811 -3612 ===========================================
- Hits 4030 1620 -2410 + Misses 1297 191 -1106 + Partials 96 0 -96

Impacted Files	Coverage Δ
go/arrow/array/table.go
go/arrow/math/uint64_amd64.go
go/arrow/internal/testing/tools/bool.go
go/arrow/internal/bitutil/bitutil.go
go/arrow/memory/memory_avx2_amd64.go
go/arrow/array/null.go
go/arrow/datatype_nested.go
go/arrow/array/string.go
go/arrow/math/uint64_avx2_amd64.go
go/arrow/array/builder.go
... and 46 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60305d5...8ccc977. Read the comment docs.

[wesm]

Are you working on this somewhere else?

[romainfrancois]

I'm not.

[romainfrancois]

I don't know what happened, but it looks I've lost some commits in that branch. this is thankfully minor.

[wesm]

Looks like you may need to git reset --hard to 8ccc977 and then rebase. It should be in your reflog

[romainfrancois]

Thanks @wesm

[wesm]

There are no tests here. What would you like to do?

[romainfrancois]

I’m still unsure how to test this. I’ll look around in pyarrow next time i look at this branch

[wesm]

Got it. I think we just generate random bytes and write them to a compressed file, then read them back in and compare

[romainfrancois]

Makes sense. IIRC I tried to serialize a record batch but that did not work out well.

[wesm]

Yeah, that requires random access. It should at least not crash, though

[wesm]

Well, to read using the *StreamReader should work. If it does not, that's buggy

[romainfrancois]

Thanks. I’ll use that in testing.

[romainfrancois]

This will likely need to be rebased once #3043 is merged

[wesm]

+1