ARROW-11417: [Integration] Add integration tests for buffer compression #9408
Conversation
There was a problem hiding this comment.
@andygrove This seems to be some automated style that got applied when running archery docker conda-integration. Is it ok?
There was a problem hiding this comment.
It's a bit of an irritant because there's a slight difference in the behaviour of the formatter in the stable vs nightly versions.
We can leave it as is on this PR. I'll make time to look into a solution.
pitrou
force-pushed
the
ARROW-11417-compression-integration
branch
from
f7c26f5 to
ad80219
Compare
github-actions
bot
added
Component: FlightRPC
Component: JavaScript
Component: Rust
labels
|
LGTM, I can't comment on JS or Rust changes. I think in the long run we'll also want proper on-demand integration tests but this is a good start. |
The tests use "gold" files generated by PyArrow 2.0.0 and are only enabled for C++.
pitrou
force-pushed
the
ARROW-11417-compression-integration
branch
from
ad80219 to
d631204
Compare
Codecov Report
@@ Coverage Diff @@
## master #9408 +/- ##
==========================================
- Coverage 82.09% 82.09% -0.01%
==========================================
Files 233 233
Lines 54375 54375
==========================================
- Hits 44639 44637 -2
- Misses 9736 9738 +2
Continue to review full report at Codecov.
|
The tests use "gold" files generated by PyArrow 2.0.0 and are only enabled for C++. Closes apache#9408 from pitrou/ARROW-11417-compression-integration Authored-by: Antoine Pitrou <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
4 participants
The tests use "gold" files generated by PyArrow 2.0.0 and are only enabled for C++.