Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0, 0.9.0
    • Component/s: Build
    • Labels:
      None

      Description

      Similar to other projects, it would be nice for Tajo to publish jars to public maven repository.

      1. TAJO-517.patch
        22 kB
        Hyunsik Choi

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-0.8.0-build #90 (See https://builds.apache.org/job/Tajo-0.8.0-build/90/)
          TAJO-517: Publish Tajo jar to a public maven repository. (hyunsik) (hyunsik: rev bede936e9ac28163eab69787c5e1c6d0180e9ca3)

          • tajo-core/pom.xml
          • tajo-project/pom.xml
          • tajo-rpc/pom.xml
          • tajo-catalog/tajo-catalog-common/pom.xml
          • tajo-client/pom.xml
          • tajo-algebra/pom.xml
          • tajo-catalog/pom.xml
          • pom.xml
          • tajo-yarn-pullserver/pom.xml
          • tajo-dist/src/main/resources/assemblies/tajo-src.xml
          • tajo-common/pom.xml
          • CHANGES
          • tajo-catalog/tajo-catalog-drivers/pom.xml
          • tajo-storage/pom.xml
          • tajo-catalog/tajo-catalog-client/pom.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-0.8.0-build #90 (See https://builds.apache.org/job/Tajo-0.8.0-build/90/ ) TAJO-517 : Publish Tajo jar to a public maven repository. (hyunsik) (hyunsik: rev bede936e9ac28163eab69787c5e1c6d0180e9ca3) tajo-core/pom.xml tajo-project/pom.xml tajo-rpc/pom.xml tajo-catalog/tajo-catalog-common/pom.xml tajo-client/pom.xml tajo-algebra/pom.xml tajo-catalog/pom.xml pom.xml tajo-yarn-pullserver/pom.xml tajo-dist/src/main/resources/assemblies/tajo-src.xml tajo-common/pom.xml CHANGES tajo-catalog/tajo-catalog-drivers/pom.xml tajo-storage/pom.xml tajo-catalog/tajo-catalog-client/pom.xml
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-master-build #197 (See https://builds.apache.org/job/Tajo-master-build/197/)
          TAJO-517: Publish Tajo jar to a public maven repository. (hyunsik) (hyunsik: rev fe8103552bb737a8085242ae67ba00732601287b)

          • tajo-catalog/pom.xml
          • tajo-catalog/tajo-catalog-drivers/pom.xml
          • tajo-rpc/pom.xml
          • CHANGES
          • tajo-catalog/tajo-catalog-common/pom.xml
          • tajo-core/pom.xml
          • tajo-storage/pom.xml
          • tajo-common/pom.xml
          • tajo-dist/src/main/resources/assemblies/tajo-src.xml
          • tajo-yarn-pullserver/pom.xml
          • pom.xml
          • tajo-project/pom.xml
          • tajo-catalog/tajo-catalog-client/pom.xml
          • tajo-client/pom.xml
          • tajo-algebra/pom.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #197 (See https://builds.apache.org/job/Tajo-master-build/197/ ) TAJO-517 : Publish Tajo jar to a public maven repository. (hyunsik) (hyunsik: rev fe8103552bb737a8085242ae67ba00732601287b) tajo-catalog/pom.xml tajo-catalog/tajo-catalog-drivers/pom.xml tajo-rpc/pom.xml CHANGES tajo-catalog/tajo-catalog-common/pom.xml tajo-core/pom.xml tajo-storage/pom.xml tajo-common/pom.xml tajo-dist/src/main/resources/assemblies/tajo-src.xml tajo-yarn-pullserver/pom.xml pom.xml tajo-project/pom.xml tajo-catalog/tajo-catalog-client/pom.xml tajo-client/pom.xml tajo-algebra/pom.xml
          Hide
          hyunsik Hyunsik Choi added a comment -

          Thanks for the review. committed it to master and branch-0.8.0.

          Show
          hyunsik Hyunsik Choi added a comment - Thanks for the review. committed it to master and branch-0.8.0.
          Hide
          jihoonson Jihoon Son added a comment -

          +1.
          I tested 'mvn clean release:prepare', and the patch seems to work well.

          Hyunsik, thanks for your efforts!

          Show
          jihoonson Jihoon Son added a comment - +1. I tested 'mvn clean release:prepare', and the patch seems to work well. Hyunsik, thanks for your efforts!
          Hide
          jihoonson Jihoon Son added a comment -

          Thanks Hyunsik!
          I'll review the patch.

          Show
          jihoonson Jihoon Son added a comment - Thanks Hyunsik! I'll review the patch.
          Hide
          tajoqa Tajo QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12641228/TAJO-517.patch
          against master revision 89e8fa3.

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

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

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

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

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

          -1 findbugs. The patch appears to introduce 201 new Findbugs (version 1.3.9) warnings.

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

          +1 core tests. The patch passed unit tests in tajo-algebra tajo-catalog/tajo-catalog-client tajo-catalog/tajo-catalog-common tajo-client tajo-common tajo-core tajo-dist tajo-rpc tajo-storage tajo-yarn-pullserver.

          Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/375//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/375//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/375//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-yarn-pullserver.html
          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/375//console

          This message is automatically generated.

          Show
          tajoqa Tajo QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12641228/TAJO-517.patch against master revision 89e8fa3. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The applied patch does not increase the total number of javadoc warnings. +1 checkstyle. The patch generated 0 code style errors. -1 findbugs. The patch appears to introduce 201 new Findbugs (version 1.3.9) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed unit tests in tajo-algebra tajo-catalog/tajo-catalog-client tajo-catalog/tajo-catalog-common tajo-client tajo-common tajo-core tajo-dist tajo-rpc tajo-storage tajo-yarn-pullserver. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/375//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/375//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/375//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-yarn-pullserver.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/375//console This message is automatically generated.
          Hide
          hyunsik Hyunsik Choi added a comment -

          This patch enables Tajo maven to create signed Jar files and to deploy them to ASF maven repository. But, I didn't fully test it because we have to wait that ASF infra team gives us the upload privilege to ASF Nexus repository (INFRA-7605). Of course, we can wait the issue resolution. But, as I mentioned in the mailing list, according to the duration of other similar issues, the resolution can be late.

          BTW, it seems that I already have the upload privilege via Nexus web UI (http://repository.apache.org). If someone review and give +1, I'll commit it and then upload Tajo jars manually to ASF nexus repository via web UI.

          Thanks,
          Hyunsik

          Show
          hyunsik Hyunsik Choi added a comment - This patch enables Tajo maven to create signed Jar files and to deploy them to ASF maven repository. But, I didn't fully test it because we have to wait that ASF infra team gives us the upload privilege to ASF Nexus repository ( INFRA-7605 ). Of course, we can wait the issue resolution. But, as I mentioned in the mailing list, according to the duration of other similar issues, the resolution can be late. BTW, it seems that I already have the upload privilege via Nexus web UI ( http://repository.apache.org ). If someone review and give +1, I'll commit it and then upload Tajo jars manually to ASF nexus repository via web UI. Thanks, Hyunsik
          Hide
          jihoonson Jihoon Son added a comment -

          Hyunsik Choi, thanks for sharing the release status and useful information!
          I hope that INFRA-7605 will be solved as soon as possible!

          Show
          jihoonson Jihoon Son added a comment - Hyunsik Choi , thanks for sharing the release status and useful information! I hope that INFRA-7605 will be solved as soon as possible!
          Hide
          hyunsik Hyunsik Choi added a comment -

          FYI, I'd like to share the progress of this work. I changed maven pom.xml file for publishing Tajo jars to Apache repository. Since its my first time for publishing maven jars, I'm still learning the way. Finally, I got the fact that we have to ask ASF infra team to enable our Nexus access. So, I requested at INFRA-7605.

          Also, I'd like to share the useful links for publishing maven jars to ASF repository. I hope that this would be helpful for other committers who will release Apache Tajo.

          Show
          hyunsik Hyunsik Choi added a comment - FYI, I'd like to share the progress of this work. I changed maven pom.xml file for publishing Tajo jars to Apache repository. Since its my first time for publishing maven jars, I'm still learning the way. Finally, I got the fact that we have to ask ASF infra team to enable our Nexus access. So, I requested at INFRA-7605 . Also, I'd like to share the useful links for publishing maven jars to ASF repository. I hope that this would be helpful for other committers who will release Apache Tajo. ASF Publishing Maven Artifacts, - http://www.apache.org/dev/publishing-maven-artifacts.html ASF Jar Repositories - http://www.apache.org/dev/repository-faq.html Apache Software Foundation Parent POM - http://maven.apache.org/pom/asf/ How To Generate PGP Signatures With Maven - https://docs.sonatype.org/display/Repository/How+To+Generate+PGP+Signatures+With+Maven

            People

            • Assignee:
              hyunsik Hyunsik Choi
              Reporter:
              hyunsik Hyunsik Choi
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development