[HIVE-13533] Remove AST dump

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.1.0
• Assignee: Jesús Camacho Rodríguez
• Reporter: Jesús Camacho Rodríguez
• Created: 18/Apr/16 09:15
• Updated: 27/Feb/24 22:24
• Resolved: 25/Apr/16 15:06

Description

For very large queries, dumping the AST can lead to OOM errors. Currently there are two places where we dump the AST:

• CalcitePlanner if we are running in DEBUG mode (line 300).
• ExplainTask if we use extended explain (line 179).

I guess the original reason to add the dump was to check whether the AST conversion from CBO was working properly, but I think we are past that stage now.

We will remove the logic to dump the AST in explain extended. For debug mode in CalcitePlanner, we will lower the level to LOG.TRACE.