[PARQUET-1385] [C++] bloom_filter-test is very slow under valgrind

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Component/s: parquet-cpp
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Wes McKinney
• Created: 17/Aug/18 06:07
• Updated: 23/Jun/24 03:30
• Resolved: 08/Nov/21 17:46

Description

This test takes ~5 minutes to run under valgrind in Travis CI

1: [==========] Running 6 tests from 6 test cases.
1: [----------] Global test environment set-up.
1: [----------] 1 test from Murmur3Test
1: [ RUN      ] Murmur3Test.TestBloomFilter
1: [       OK ] Murmur3Test.TestBloomFilter (19 ms)
1: [----------] 1 test from Murmur3Test (34 ms total)
1: 
1: [----------] 1 test from ConstructorTest
1: [ RUN      ] ConstructorTest.TestBloomFilter
1: [       OK ] ConstructorTest.TestBloomFilter (101 ms)
1: [----------] 1 test from ConstructorTest (101 ms total)
1: 
1: [----------] 1 test from BasicTest
1: [ RUN      ] BasicTest.TestBloomFilter
1: [       OK ] BasicTest.TestBloomFilter (49 ms)
1: [----------] 1 test from BasicTest (49 ms total)
1: 
1: [----------] 1 test from FPPTest
1: [ RUN      ] FPPTest.TestBloomFilter
1: [       OK ] FPPTest.TestBloomFilter (308731 ms)
1: [----------] 1 test from FPPTest (308741 ms total)
1: 
1: [----------] 1 test from CompatibilityTest
1: [ RUN      ] CompatibilityTest.TestBloomFilter
1: [       OK ] CompatibilityTest.TestBloomFilter (62 ms)
1: [----------] 1 test from CompatibilityTest (62 ms total)
1: 
1: [----------] 1 test from OptimalValueTest
1: [ RUN      ] OptimalValueTest.TestBloomFilter
1: [       OK ] OptimalValueTest.TestBloomFilter (27 ms)
1: [----------] 1 test from OptimalValueTest (27 ms total)
1: 
1: [----------] Global test environment tear-down
1: [==========] 6 tests from 6 test cases ran. (309081 ms total)
1: [  PASSED  ] 6 tests.

Either we should change the FPPTest parameters to be faster, or we should not run that test when using valrind

Issue Links

• is fixed by: PARQUET-1382 [C++] Prepare for arrow::test namespace removal
• links to: GitHub Pull Request #489