Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-281

'mvn package -Pdist' generates duplicate Tajo jar files

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.2-incubating, 0.8.0
    • Component/s: Build
    • Labels:
      None

      Description

      See the title. We need to remove duplicate Tajo jar files from a package generated by 'mvn package -Pdist'.

      1. TAJO-281.patch
        2 kB
        Hyunsik Choi

        Activity

        Hide
        hyunsik Hyunsik Choi added a comment -

        I've committed this patch to branch-0.2 and master branch. Thanks for the review!

        Show
        hyunsik Hyunsik Choi added a comment - I've committed this patch to branch-0.2 and master branch. Thanks for the review!
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-trunk-postcommit #538 (See https://builds.apache.org/job/Tajo-trunk-postcommit/538/)
        TAJO-281: 'mvn package -Pdist' generates duplicate Tajo jar files. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=c0d71913e746f2c930425837a6516cf49d8f80f7)

        • tajo-core/pom.xml
        • tajo-dist/pom.xml
        • CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-trunk-postcommit #538 (See https://builds.apache.org/job/Tajo-trunk-postcommit/538/ ) TAJO-281 : 'mvn package -Pdist' generates duplicate Tajo jar files. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=c0d71913e746f2c930425837a6516cf49d8f80f7 ) tajo-core/pom.xml tajo-dist/pom.xml CHANGES.txt
        Hide
        hyunsik Hyunsik Choi added a comment -

        I've also tested this patch on a real cluster with TPC-H Q1. It works well.

        Show
        hyunsik Hyunsik Choi added a comment - I've also tested this patch on a real cluster with TPC-H Q1. It works well.
        Hide
        jhkim Jinho Kim added a comment - - edited

        +1
        Looks great for me. I've verified 'mvn clean package -Pdist' on master, branch-0.2.

        Show
        jhkim Jinho Kim added a comment - - edited +1 Looks great for me. I've verified 'mvn clean package -Pdist' on master, branch-0.2.
        Hide
        hyunsik Hyunsik Choi added a comment -

        I've fixed this problem, and I've verified 'mvn clean install' and the test on a local cluster.

        Show
        hyunsik Hyunsik Choi added a comment - I've fixed this problem, and I've verified 'mvn clean install' and the test on a local cluster.

          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