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

Memoize toStringTree() for ASTNode

Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Won't Fix
    • 1.0.0, 1.1.0, 1.2.0
    • None
    • Query Planning
    • 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.3.patch
          1 kB
          Ashutosh Chauhan
        2. HIVE-11281.2.patch
          2 kB
          Ashutosh Chauhan
        3. HIVE-11281.patch
          2 kB
          Ashutosh Chauhan

        Issue Links

        is part of

        Task - A task that needs to be done. HIVE-11315 Compilation performance issues for large queries

        • Major - Major loss of function.
        • Resolved

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ashutoshc Ashutosh Chauhan Assign to me
            ashutoshc Ashutosh Chauhan
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment