[IMPALA-6738] further refactor how concurrent_select's args get plumbed through

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Impala 2.12.0
• Component/s: Infrastructure
• Target Version: Product Backlog
• Epic Color: ghx-label-2
• Assignee: Unassigned
• Reporter: Michael Brown
• Created: 26/Mar/18 17:24
• Updated: 19/May/20 20:02
• Resolved: 19/May/20 20:02

Description

The stress test has many commandline arguments, and many of them need to be plumbed through to different parts of the code, including running the binary search and running the stress test proper. It's getting messy to deal with all these, so we need a saner way to do this. I'm working on a patch that should start this, but the patch won't complete the effort.

Some places this is gross:

populate_runtime_info()
load_random_queries_and_populate_runtime_info()
populate_runtime_info_for_random_queries()
populate_all_queries()