add more integration test for parquet bloom filter round trip tests#3210
Conversation
a64f91c to
f6d1f37
Compare
f6d1f37 to
4121286
Compare
alamb
left a comment
There was a problem hiding this comment.
Thank you @jimexist
I also double checked the new test runs in CI correctly and in a reasonable amount of time:
https://github.com/apache/arrow-rs/actions/runs/3781373365/jobs/6428207049
============================= test session starts ==============================
platform linux -- Python 3.10.9, pytest-7.2.0, pluggy-1.0.0 -- /opt/hostedtoolcache/Python/3.10.9/x64/bin/python
cachedir: .pytest_cache
rootdir: /home/runner/work/arrow-rs/arrow-rs/parquet/pytest
collecting ... collected 4 items
test_parquet_integration.py::TestParquetIntegration::test_pyspark_bloom_filter[1] PASSED [ 25%]
test_parquet_integration.py::TestParquetIntegration::test_pyspark_bloom_filter[10] PASSED [ 50%]
test_parquet_integration.py::TestParquetIntegration::test_bloom_filter_round_trip[1] PASSED [ 75%]
test_parquet_integration.py::TestParquetIntegration::test_bloom_filter_round_trip[10] PASSED [100%]
👍
|
Benchmark runs are scheduled for baseline = 513d543 and contender = 99a20dd. 99a20dd is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
Closes #.
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?