[CALCITE-2878] Avoid use of new RuntimeException(e) in tests

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.19.0
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Vladimir Sitnikov
• Created: 27/Feb/19 07:03
• Updated: 25/Mar/19 00:11
• Resolved: 27/Feb/19 07:32

Description

new RuntimeException(e) adds very little info in tests, and it makes stacktraces harder to read.
So it makes sense to just sneaky-throw in those cases, especially for test purposes

Issue Links

• links to: GitHub Pull Request #1042