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

Memoize toStringTree() for ASTNode

Log workAgile BoardRank to TopRank to BottomVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.0.0, 1.1.0, 1.2.0
    • Fix Version/s: None
    • Component/s: Query Planning
    • Labels:
      None

      Description

      Currently it defaults to BaseTree::toStringTree() which always computes it recursively. For very large trees it becomes expensive. On one particular test query compilation time went down from 55 seconds to 2 seconds.

        Attachments

        1. HIVE-11281.2.patch
          2 kB
          Ashutosh Chauhan
        2. HIVE-11281.3.patch
          1 kB
          Ashutosh Chauhan
        3. HIVE-11281.patch
          2 kB
          Ashutosh Chauhan

        Issue Links

          Activity

          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

            • Assignee:
              ashutoshc Ashutosh Chauhan Assign to me
              Reporter:
              ashutoshc Ashutosh Chauhan

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment