Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-21346 Refactor Hive DDL
  3. HIVE-22276

Break up DDLSemanticAnalyzer - extract View related analyzers

Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersConvert to IssueMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • None

    Description

      DDLSemanticAnalyzer is a huge class, more than 4000 lines long. The goal is to refactor it in order to have everything cut into more handleable classes under the packageĀ  org.apache.hadoop.hive.ql.exec.ddl:

      • have a separate class for each analyzers
      • have a package for each operation, containing an analyzer, a description, and an operation, so the amount of classes under a package is more manageable

      Step #6: extract all the vire related analyzers from DDLSemanticAnalyzer, and move them under the new package.

      Attachments

        1. HIVE-22276.01.patch
          57 kB
          Miklos Gergely
        2. HIVE-22276.02.patch
          66 kB
          Miklos Gergely
        3. HIVE-22276.03.patch
          69 kB
          Miklos Gergely

        Activity

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

          People

            mgergely Miklos Gergely Assign to me
            mgergely Miklos Gergely
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 1.5h
              1.5h

              Slack

                Issue deployment