Uploaded image for project: 'Apache Ozone'
  1. Apache Ozone
  2. HDDS-9136

Rename should throw exception upon error

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.4.0
    • OFS

    Description

      We encountered an interesting bug with Impala/Hive where sometimes Impala shows double the number of results.

      Here's what happened.

       

      1. Impala issues DROP TABLE, which moves the table to the trash directory.
      2. At the same time, Ozone's trash policy moves the trash directory .Trash/Current to .Trash/<timestamp>.
      3. HMS checks .Trash/Current, thinking it exists, and moves the table directory under .Trash/Current, which is being moved away.
      4. The rename fails, Ozone rename returns false instead of exception. The trash policy doesn't check return value and simply assumes it succeeded.
      5. HMS fails to delete files, so they’re left behind in a standard location for this table.
      6. Impala creates a new table. This is purely a metadata operation, it doesn’t do anything in the filesystem. When a table is partitioned, partition existence is metadata. The table starts with 0 partitions.
      7. Because there are 0 partitions, when select count is first run Impala sees 0 partitions so it doesn’t look for files. It immediately returns 0 rows.
      8. INSERT OVERWRITE adds a partition. That partition location is in a standard location, which happens to already have files. OVERWRITE does nothing because previously Impala had no partitions (this might be considered a bug, weird corner case). INSERT adds new files with unique names to the partition directory. We now have double the number of files.
      9. Next select count reads all the files in the partition.

       

      Michael Smith 

      Attachments

        Activity

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

          People

            ashishk Ashish Kumar
            weichiu Wei-Chiu Chuang
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment