Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-13533

Remove AST dump

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.1.0
    • 2.1.0
    • None
    • None

    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.

      Attachments

        1. HIVE-13533.patch
          17 kB
          Jesus Camacho Rodriguez
        2. HIVE-13533.patch
          17 kB
          Jesus Camacho Rodriguez

        Activity

          People

            jcamachorodriguez Jesus Camacho Rodriguez
            jcamachorodriguez Jesus Camacho Rodriguez
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: