[ARROW-7552] [C++] TestSlowInputStream is flaky

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.16.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/23812
• Assignee: Antoine Pitrou
• Reporter: Neal Richardson
• Created: 11/Jan/20 01:28
• Updated: 11/Jan/23 07:54
• Resolved: 14/Jan/20 00:23

Description

See https://github.com/apache/arrow/pull/6160/checks?check_run_id=384146741#step:5:1556 for example

[ RUN      ] TestSlowInputStream.Basics
/arrow/cpp/src/arrow/io/memory_test.cc:308: Failure
Expected: (dt) < (latency * 3), actual: 4.96068 vs 1.8
[  FAILED  ] TestSlowInputStream.Basics (4961 ms)
[----------] 1 test from TestSlowInputStream (4961 ms total)

Tests that rely on timing are pretty tough to do on public CI. We should consider moving this somewhere that doesn't run on CI.

Issue Links

• links to: GitHub Pull Request #6173