[DRILL-4911] SimpleParallelizer should avoid plan serialization for logging purpose when debug logging is not enabled.

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Component/s: Query Planning & Optimization
• Assignee: Jinfeng Ni
• Reporter: Jinfeng Ni
• Reviewer: Krystal
• Created: 28/Sep/16 17:47
• Updated: 08/Nov/16 17:00
• Resolved: 06/Oct/16 05:11

Description

SimpleParallelizer and subclass have code to put serialized plan in debug log. However, in case when the physical plan fragment is very large (table is huge with large # of partitions, files, etc), the overhead of serialization is not negligible. The code seems to always do serialization, regardless whether the debug flag is turned on.

To reduce the overhead, SimpleParallelizer and subclass should only do plan serialization for logging purpose when debug logging is turned on.

Issue Links

• links to: GitHub Pull Request #599