[ARROW-17109] [C++] Clean up includes in exec_plan.h

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Not A Problem
• Component/s: C++
• Labels: good-first-issue good-second-issue query-engine
• External issue URL: https://github.com/apache/arrow/issues/32414
• Assignee: Unassigned
• Reporter: David Li
• Created: 18/Jul/22 15:05
• Updated: 11/Jan/23 11:48
• Resolved: 20/Dec/22 13:12

Description

Notably, it includes logging.h transitively via exec/util.h which we should avoid. We should perhaps add to/create an arrow/compute/exec/type_fwd.h