Details

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

    Description

      Refer - https://cwiki.apache.org/confluence/display/Hive/LanguageManual+DDL#LanguageManualDDL-AlterTable
      Support Alter table with
      table properties,
      storage,
      change serde, serde props,
      Comments
      Clustering and bucketing

      Attachments

        1. ATLAS-522.1.patch
          22 kB
          Suma Shivaprasad
        2. ATLAS-522.2.patch
          23 kB
          Suma Shivaprasad
        3. ATLAS-522.patch
          23 kB
          Suma Shivaprasad

        Activity

          suma.shivaprasad Suma Shivaprasad added a comment - https://reviews.apache.org/r/44600/
          atlasqa ATLAS QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12792379/ATLAS-522.patch
          against master revision 1610791.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 1 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 checkstyle. The patch generated 0 code style errors.

          -1 findbugs. The patch appears to introduce 333 new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in :
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.EntityJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.MetadataDiscoveryJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.RexsterGraphJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.AdminJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.TypesJerseyResourceIT
          ./webapp/target/failsafe-reports/junitreports/TEST-org.apache.atlas.examples.QuickStartIT

          Test results: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsnotification.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningscommon.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsclient.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningstitan.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningstypesystem.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsrepository.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningswebapp.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsfalcon-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningshive-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsstorm-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningssqoop-bridge.html
          Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//console

          This message is automatically generated.

          atlasqa ATLAS QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12792379/ATLAS-522.patch against master revision 1610791. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 checkstyle. The patch generated 0 code style errors. -1 findbugs . The patch appears to introduce 333 new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in : ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.EntityJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.MetadataDiscoveryJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.RexsterGraphJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.AdminJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.TypesJerseyResourceIT ./webapp/target/failsafe-reports/junitreports/TEST-org.apache.atlas.examples.QuickStartIT Test results: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsnotification.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningscommon.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsclient.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningstitan.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningstypesystem.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsrepository.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningswebapp.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsfalcon-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningshive-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningsstorm-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//artifact/patchprocess/newPatchFindbugsWarningssqoop-bridge.html Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/60//console This message is automatically generated.

          Removed unnecessary table load since that was already happening in createOrUpdateEntities and added missing break

          suma.shivaprasad Suma Shivaprasad added a comment - Removed unnecessary table load since that was already happening in createOrUpdateEntities and added missing break
          atlasqa ATLAS QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12792419/ATLAS-522.1.patch
          against master revision 1610791.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 1 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 checkstyle. The patch generated 0 code style errors.

          -1 findbugs. The patch appears to introduce 332 new Findbugs (version 2.0.3) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in :
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.EntityJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.MetadataDiscoveryJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.RexsterGraphJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.AdminJerseyResourceIT
          ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.TypesJerseyResourceIT

          Test results: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsnotification.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningscommon.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsclient.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningstitan.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningstypesystem.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsrepository.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningswebapp.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsfalcon-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningshive-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsstorm-bridge.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningssqoop-bridge.html
          Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//console

          This message is automatically generated.

          atlasqa ATLAS QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12792419/ATLAS-522.1.patch against master revision 1610791. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 checkstyle. The patch generated 0 code style errors. -1 findbugs . The patch appears to introduce 332 new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in : ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.EntityJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.MetadataDiscoveryJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.RexsterGraphJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.AdminJerseyResourceIT ./webapp/test-output/junitreports/TEST-org.apache.atlas.web.resources.TypesJerseyResourceIT Test results: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsnotification.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningscommon.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsclient.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningstitan.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningstypesystem.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsrepository.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningswebapp.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsfalcon-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningshive-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningsstorm-bridge.html Findbugs warnings: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//artifact/patchprocess/newPatchFindbugsWarningssqoop-bridge.html Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/61//console This message is automatically generated.

          Fixed review comments

          suma.shivaprasad Suma Shivaprasad added a comment - Fixed review comments
          shwethags Shwetha GS added a comment -

          Committed to master. Thanks Suma

          shwethags Shwetha GS added a comment - Committed to master. Thanks Suma

          People

            suma.shivaprasad Suma Shivaprasad
            suma.shivaprasad Suma Shivaprasad
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: