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

Optimize AcidUtils.getLogicalLength()

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.0.0
    • 4.0.0-alpha-1
    • Transactions
    • None
    • Hide
      I messed up the commit msg for this. The Jira number is correct, but the bug title is from another issue

      {noformat}
      commit 07b76f413cb174413f0530a6aae5ae442a301b46
      Author: Eugene Koifman <ekoifman@apache.org>
      Date: Thu Feb 7 09:49:19 2019 -0800

          HIVE-21177: ACID: When there are no delete deltas skip finding min max keys (Eugene Koifman, reviewed by Prasanth Jayachandran)
      {noformat}
      Show
      I messed up the commit msg for this. The Jira number is correct, but the bug title is from another issue {noformat} commit 07b76f413cb174413f0530a6aae5ae442a301b46 Author: Eugene Koifman < ekoifman@apache.org > Date: Thu Feb 7 09:49:19 2019 -0800      HIVE-21177 : ACID: When there are no delete deltas skip finding min max keys (Eugene Koifman, reviewed by Prasanth Jayachandran) {noformat}

    Description

      AcidUtils.getLogicalLength() - tries look for the side file OrcAcidUtils.getSideFile() on the file system even when the file couldn't possibly be there, e.g. when the path is delta_x_x or base_x. It could only be there in delta_x_y, x != y.

      Attachments

        1. HIVE-21177.01.patch
          10 kB
          Eugene Koifman
        2. HIVE-21177.02.patch
          19 kB
          Eugene Koifman
        3. HIVE-21177.03.patch
          45 kB
          Eugene Koifman

        Issue Links

          is required by

          Improvement - An improvement or enhancement to an existing feature or task. HIVE-21225 ACID: getAcidState() should cache a recursive dir listing locally

          • Major - Major loss of function.
          • Closed

          Activity

            People

              ekoifman Eugene Koifman
              ekoifman Eugene Koifman
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: