Hive
  1. Hive
  2. HIVE-239

alter table repalce columns allows adding of new columns with the same name as that of a partition column which corrupts metadata

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.3.0
    • Component/s: Metastore
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      there is no way for user to revert back and this check should be done in alter table function.

      1. hive-239.3.patch
        7 kB
        Prasad Chakka
      2. hive-239.2.patch
        7 kB
        Prasad Chakka
      3. hive-239.patch
        6 kB
        Prasad Chakka
      4. hive-239.patch
        6 kB
        Prasad Chakka

        Activity

        Hide
        Prasad Chakka added a comment -

        previous patch was relative ql instead of hive/trunk

        Show
        Prasad Chakka added a comment - previous patch was relative ql instead of hive/trunk
        Hide
        Ashish Thusoo added a comment -

        Looks fine to me.

        The error message though has a misspelling for column...

        Can you fix that. I will check it in once that is done...

        Show
        Ashish Thusoo added a comment - Looks fine to me. The error message though has a misspelling for column... Can you fix that. I will check it in once that is done...
        Hide
        Prasad Chakka added a comment -

        fixed the spelling mistake

        Show
        Prasad Chakka added a comment - fixed the spelling mistake
        Hide
        Zheng Shao added a comment -

        +1

        Show
        Zheng Shao added a comment - +1
        Hide
        Ashish Thusoo added a comment -

        some of the tests failed due to the change in the error message. Can you recapture and upload a new patch.

        Show
        Ashish Thusoo added a comment - some of the tests failed due to the change in the error message. Can you recapture and upload a new patch.
        Hide
        Prasad Chakka added a comment -

        updated two test outputs to match the incorrect spelling

        Show
        Prasad Chakka added a comment - updated two test outputs to match the incorrect spelling
        Hide
        Ashish Thusoo added a comment -

        committed. Thanks Prasad!!

        Show
        Ashish Thusoo added a comment - committed. Thanks Prasad!!

          People

          • Assignee:
            Prasad Chakka
            Reporter:
            Prasad Chakka
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development