Uploaded image for project: '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
        • Status: Closed
        • Priority: 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. Text File
            hive-239.2.patch
            7 kB
            Prasad Chakka
          2. Text File
            hive-239.3.patch
            7 kB
            Prasad Chakka
          3. Text File
            hive-239.patch
            6 kB
            Prasad Chakka
          4. Text File
            hive-239.patch
            6 kB
            Prasad Chakka

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            prasadc Prasad Chakka added a comment -

            previous patch was relative ql instead of hive/trunk

            Show
            prasadc Prasad Chakka added a comment - previous patch was relative ql instead of hive/trunk
            Hide
            Permalink
            athusoo 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
            athusoo 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
            Permalink
            prasadc Prasad Chakka added a comment -

            fixed the spelling mistake

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

            +1

            Show
            zshao Zheng Shao added a comment - +1
            Hide
            Permalink
            athusoo 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
            athusoo 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
            Permalink
            prasadc Prasad Chakka added a comment -

            updated two test outputs to match the incorrect spelling

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

            committed. Thanks Prasad!!

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development