Details

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

      Description

      Maven pom files include many dependencies. Some of the dependencies are not necessary, and others should be changed as provided. The main objective of this issue is to clean up maven dependencies.

      1. TAJO-753_20140415_16:58:08.patch
        111 kB
        Jinho Kim
      2. TAJO-753_20140414_21:01:38.patch
        100 kB
        Jinho Kim
      3. TAJO-753_20140414_15:25:18.patch
        91 kB
        Jinho Kim
      4. TAJO-753.patch
        77 kB
        Jinho Kim

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-master-build #173 (See https://builds.apache.org/job/Tajo-master-build/173/)
          TAJO-753: Clean up of maven dependencies. (jinho) (jinossy: rev 1d24a25acbf523172127cd6456b231370db1ce11)

          • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogConstants.java
          • tajo-core/tajo-core-backend/pom.xml
          • tajo-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/listener/FileCloseListener.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/listener/FileCloseListener.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java
          • tajo-rpc/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/StorageConstants.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/test/java/org/apache/tajo/catalog/store/TestHCatalogStore.java
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestNullValues.java
          • tajo-catalog/tajo-catalog-client/pom.xml
          • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
          • tajo-storage/pom.xml
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestCTASQuery.java
          • tajo-client/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java
          • tajo-catalog/tajo-catalog-common/pom.xml
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/client/TestDDLBuilder.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/benchmark/TPCH.java
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanner.java
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestStorageManager.java
          • tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
          • CHANGES.txt
          • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.3.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TpchTestBase.java
          • tajo-jdbc/pom.xml
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestMergeScanner.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-core/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/AbstractStorageManager.java
          • tajo-common/src/test/java/org/apache/tajo/datum/TestInt2Datum.java
          • tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
          • tajo-project/pom.xml
          • tajo-common/pom.xml
          • tajo-rpc/src/main/java/org/apache/tajo/rpc/ServerCallable.java
          • tajo-algebra/pom.xml
          • tajo-catalog/tajo-catalog-server/pom.xml
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestStorages.java
          • tajo-catalog/tajo-catalog-drivers/pom.xml
          • tajo-core/tajo-core-pullserver/pom.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #173 (See https://builds.apache.org/job/Tajo-master-build/173/ ) TAJO-753 : Clean up of maven dependencies. (jinho) (jinossy: rev 1d24a25acbf523172127cd6456b231370db1ce11) tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogConstants.java tajo-core/tajo-core-backend/pom.xml tajo-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/listener/FileCloseListener.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/listener/FileCloseListener.java tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java tajo-rpc/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/StorageConstants.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/test/java/org/apache/tajo/catalog/store/TestHCatalogStore.java tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestNullValues.java tajo-catalog/tajo-catalog-client/pom.xml tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java tajo-storage/pom.xml tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestCTASQuery.java tajo-client/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java tajo-catalog/tajo-catalog-common/pom.xml tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/client/TestDDLBuilder.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/benchmark/TPCH.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanner.java tajo-storage/src/test/java/org/apache/tajo/storage/TestStorageManager.java tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java CHANGES.txt tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.3.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TpchTestBase.java tajo-jdbc/pom.xml tajo-storage/src/test/java/org/apache/tajo/storage/TestMergeScanner.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-core/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/AbstractStorageManager.java tajo-common/src/test/java/org/apache/tajo/datum/TestInt2Datum.java tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-project/pom.xml tajo-common/pom.xml tajo-rpc/src/main/java/org/apache/tajo/rpc/ServerCallable.java tajo-algebra/pom.xml tajo-catalog/tajo-catalog-server/pom.xml tajo-storage/src/test/java/org/apache/tajo/storage/TestStorages.java tajo-catalog/tajo-catalog-drivers/pom.xml tajo-core/tajo-core-pullserver/pom.xml
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-0.8.0-build #72 (See https://builds.apache.org/job/Tajo-0.8.0-build/72/)
          TAJO-753: Clean up of maven dependencies. (jinho) (jinossy: rev 646cf1ade76379f62d34657d032d6ee1ef8088f0)

          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestCTASQuery.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/listener/FileCloseListener.java
          • tajo-catalog/tajo-catalog-common/pom.xml
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/benchmark/TPCH.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java
          • tajo-core/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/AbstractStorageManager.java
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/client/TestDDLBuilder.java
          • tajo-common/src/test/java/org/apache/tajo/datum/TestInt2Datum.java
          • tajo-rpc/pom.xml
          • tajo-jdbc/pom.xml
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TpchTestBase.java
          • CHANGES.txt
          • tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-core/tajo-core-backend/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/test/java/org/apache/tajo/catalog/store/TestHCatalogStore.java
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestStorages.java
          • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml
          • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.3.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java
          • tajo-catalog/tajo-catalog-server/pom.xml
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestStorageManager.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/StorageConstants.java
          • tajo-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestNullValues.java
          • tajo-project/pom.xml
          • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogConstants.java
          • tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/listener/FileCloseListener.java
          • tajo-algebra/pom.xml
          • tajo-common/pom.xml
          • tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
          • tajo-rpc/src/main/java/org/apache/tajo/rpc/ServerCallable.java
          • tajo-storage/src/test/java/org/apache/tajo/storage/TestMergeScanner.java
          • tajo-client/pom.xml
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanner.java
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
          • tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java
          • tajo-core/tajo-core-pullserver/pom.xml
          • tajo-storage/pom.xml
          • tajo-catalog/tajo-catalog-client/pom.xml
          • tajo-catalog/tajo-catalog-drivers/pom.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-0.8.0-build #72 (See https://builds.apache.org/job/Tajo-0.8.0-build/72/ ) TAJO-753 : Clean up of maven dependencies. (jinho) (jinossy: rev 646cf1ade76379f62d34657d032d6ee1ef8088f0) tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestCTASQuery.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/listener/FileCloseListener.java tajo-catalog/tajo-catalog-common/pom.xml tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/benchmark/TPCH.java tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java tajo-core/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/AbstractStorageManager.java tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/client/TestDDLBuilder.java tajo-common/src/test/java/org/apache/tajo/datum/TestInt2Datum.java tajo-rpc/pom.xml tajo-jdbc/pom.xml tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/TpchTestBase.java CHANGES.txt tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.4.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-core/tajo-core-backend/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/test/java/org/apache/tajo/catalog/store/TestHCatalogStore.java tajo-storage/src/test/java/org/apache/tajo/storage/TestStorages.java tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.3.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java tajo-catalog/tajo-catalog-server/pom.xml tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/FadvisedFileRegionWrapper.java tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java tajo-storage/src/test/java/org/apache/tajo/storage/TestStorageManager.java tajo-storage/src/main/java/org/apache/tajo/storage/StorageConstants.java tajo-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestNullValues.java tajo-project/pom.xml tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogConstants.java tajo-core/tajo-core-pullserver/src/main/hadoop-2.2.0/org/apache/tajo/pullserver/listener/FileCloseListener.java tajo-algebra/pom.xml tajo-common/pom.xml tajo-core/tajo-core-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java tajo-rpc/src/main/java/org/apache/tajo/rpc/ServerCallable.java tajo-storage/src/test/java/org/apache/tajo/storage/TestMergeScanner.java tajo-client/pom.xml tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanner.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java tajo-core/tajo-core-pullserver/pom.xml tajo-storage/pom.xml tajo-catalog/tajo-catalog-client/pom.xml tajo-catalog/tajo-catalog-drivers/pom.xml
          Hide
          jhkim Jinho Kim added a comment -

          This issue got +1 from reviewboard.
          I've just committed it to both master branch and branch-0.8.0.

          Show
          jhkim Jinho Kim added a comment - This issue got +1 from reviewboard. I've just committed it to both master branch and branch-0.8.0.
          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/12640217/TAJO-753_20140415_16%3A58%3A08.patch
          against master revision 06a1496.

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

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

          +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 210 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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-jdbc tajo-rpc tajo-storage.

          Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/342//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-pullserver.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-backend.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-jdbc.html
          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/342//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/12640217/TAJO-753_20140415_16%3A58%3A08.patch against master revision 06a1496. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 10 new or modified test files. +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 210 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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-jdbc tajo-rpc tajo-storage. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/342//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-pullserver.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-backend.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/342//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-jdbc.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/342//console This message is automatically generated.
          Hide
          jhkim Jinho Kim added a comment -

          Updated the review request against branch master in reviewboard
          https://reviews.apache.org/r/20304/

          Show
          jhkim Jinho Kim added a comment - Updated the review request against branch master in reviewboard https://reviews.apache.org/r/20304/
          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/12640061/TAJO-753_20140414_21%3A01%3A38.patch
          against master revision 06a1496.

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

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

          +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 cause Findbugs (version 1.3.9) to fail.

          +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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-jdbc tajo-rpc tajo-storage.

          Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/340//testReport/
          Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/340//findbugsResult
          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/340//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/12640061/TAJO-753_20140414_21%3A01%3A38.patch against master revision 06a1496. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 10 new or modified test files. +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 cause Findbugs (version 1.3.9) to fail. +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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-jdbc tajo-rpc tajo-storage. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/340//testReport/ Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/340//findbugsResult Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/340//console This message is automatically generated.
          Hide
          jhkim Jinho Kim added a comment -

          Additionally, I've fix the build on hadoop-2.4.0

          Show
          jhkim Jinho Kim added a comment - Additionally, I've fix the build on hadoop-2.4.0
          Hide
          jhkim Jinho Kim added a comment -

          Updated the review request against branch master in reviewboard
          https://reviews.apache.org/r/20304/

          Show
          jhkim Jinho Kim added a comment - Updated the review request against branch master in reviewboard https://reviews.apache.org/r/20304/
          Hide
          jhkim Jinho Kim added a comment -

          I missed tajo-core-backend project. I will add it

          Show
          jhkim Jinho Kim added a comment - I missed tajo-core-backend project. I will add it
          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/12640032/TAJO-753_20140414_15%3A25%3A18.patch
          against master revision 8982684.

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

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

          +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 cause Findbugs (version 1.3.9) to fail.

          +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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-rpc tajo-storage.

          Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/336//testReport/
          Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/336//findbugsResult
          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/336//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/12640032/TAJO-753_20140414_15%3A25%3A18.patch against master revision 8982684. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 10 new or modified test files. +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 cause Findbugs (version 1.3.9) to fail. +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-catalog/tajo-catalog-drivers/tajo-hcatalog tajo-catalog/tajo-catalog-server tajo-client tajo-common tajo-core/tajo-core-backend tajo-core/tajo-core-pullserver tajo-rpc tajo-storage. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/336//testReport/ Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/336//findbugsResult Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/336//console This message is automatically generated.
          Hide
          jhkim Jinho Kim added a comment -

          Updated the review request against branch master in reviewboard

          Show
          jhkim Jinho Kim added a comment - Updated the review request against branch master in reviewboard
          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/12640026/TAJO-753.patch
          against master revision 8982684.

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

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

          -1 javac. The patch appears to cause the build to fail.

          Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/335//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/12640026/TAJO-753.patch against master revision 8982684. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 10 new or modified test files. -1 javac. The patch appears to cause the build to fail. Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/335//console This message is automatically generated.
          Hide
          jhkim Jinho Kim added a comment -

          Created a review request against branch master in reviewboard
          https://reviews.apache.org/r/20304/

          Show
          jhkim Jinho Kim added a comment - Created a review request against branch master in reviewboard https://reviews.apache.org/r/20304/
          Hide
          jhkim Jinho Kim added a comment -

          Thanks, Hyunsik
          If there are no suggestion, I'll start this issue after a while.

          Show
          jhkim Jinho Kim added a comment - Thanks, Hyunsik If there are no suggestion, I'll start this issue after a while.
          Hide
          hyunsik Hyunsik Choi added a comment -

          The target users of tajo jars are usually developers to Tajo applcations. Tajo runs on hadoop 2.2.0, 2.3.0 or possibly higher, which will be released future. If Tajo maven pom fixes hadoop version, it will be hard for developers to make use Tajo maven pom. So, I'd like to suggest that we should change hadoop's dependencies as provided.

          any other suggestion?

          Show
          hyunsik Hyunsik Choi added a comment - The target users of tajo jars are usually developers to Tajo applcations. Tajo runs on hadoop 2.2.0, 2.3.0 or possibly higher, which will be released future. If Tajo maven pom fixes hadoop version, it will be hard for developers to make use Tajo maven pom. So, I'd like to suggest that we should change hadoop's dependencies as provided . any other suggestion?

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development