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

Remove unnecessary attach-jdbc-driver execution from package/pom.xml

Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.2.0
    • Build Infrastructure
    • None

    Description

      Looks like build-helper-maven-plugin block which has execution attach-jdbc-driver is not needed in package/pom.xml

      package/pom.xml has maven-dependency-plugin which copies hive-jdbc-standalone to project.build.directory

      I removed build-helper-maven-plugin block and rebuilt hive
      hive-jdbc-standalone.jar is still placed to project.build.directory

      $ mvn clean install -Phadoop-2 -Pdist -DskipTests
      
      $ find . -name "apache-hive*jdbc.jar" -exec ls -la {} \;
      16844023 Feb  6 17:45 ./packaging/target/apache-hive-1.2.0-SNAPSHOT-jdbc.jar
      
      $ find . -name "hive-jdbc*standalone.jar" -exec ls -la {} \;
      16844023 Feb  6 17:45 ./packaging/target/apache-hive-1.2.0-SNAPSHOT-bin/apache-hive-1.2.0-SNAPSHOT-bin/lib/hive-jdbc-1.2.0-SNAPSHOT-standalone.jar
      16844023 Feb  6 17:45 ./jdbc/target/hive-jdbc-1.2.0-SNAPSHOT-standalone.jar
      

      Attachments

        1. HIVE-9607.1.patch
          1 kB
          Alexander Pivovarov

        Activity

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

          People

            apivovarov Alexander Pivovarov Assign to me
            apivovarov Alexander Pivovarov
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment