You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by "Jesus Camacho Rodriguez (JIRA)" <ji...@apache.org> on 2016/04/18 11:15:25 UTC

[jira] [Created] (HIVE-13533) Remove AST dump

Jesus Camacho Rodriguez created HIVE-13533:
----------------------------------------------

             Summary: Remove AST dump
                 Key: HIVE-13533
                 URL: https://issues.apache.org/jira/browse/HIVE-13533
             Project: Hive
          Issue Type: Bug
    Affects Versions: 2.1.0
            Reporter: Jesus Camacho Rodriguez
            Assignee: Jesus Camacho Rodriguez


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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)