[ARROW-14641] [C++][Compute] Remove print statements from unit tests

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 6.0.0
• Fix Version/s: 7.0.0
• Component/s: C++
• Labels: beginner-friendly pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/30184
• Assignee: Dhruv Vats
• Reporter: Ben Kietzman
• Created: 09/Nov/21 15:49
• Updated: 11/Jan/23 08:41
• Resolved: 20/Nov/21 03:19

Description

Unit tests should avoid unconditional print statements, preferring SCOPED_TRACE, ARROW_SCOPED_TRACE, or on-fail extra messages like ASSERT_EQ(1, 2) << "extra here";. There are some in the hash join unit tests

Issue Links

• links to: GitHub Pull Request #11663