Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: Build
    • Labels:
      None

      Description

      if a protocol buffer idl modifed, target directories should be cleaned

      [INFO] Scanning for projects...
      [INFO]                                                                         
      [INFO] ------------------------------------------------------------------------
      [INFO] Building Tajo Catalog Common 0.9.0-SNAPSHOT
      [INFO] ------------------------------------------------------------------------
      [INFO] 
      [INFO] --- maven-antrun-plugin:1.6:run (default) @ tajo-catalog-common ---
      [INFO] Executing tasks
      
      main:
      [INFO] Executed tasks
      [INFO] 
      [INFO] --- maven-antrun-plugin:1.6:run (create-protobuf-generated-sources-directory) @ tajo-catalog-common ---
      [INFO] Executing tasks
      
      main:
      [INFO] Executed tasks
      [INFO] 
      [INFO] --- exec-maven-plugin:1.2:exec (generate-sources) @ tajo-catalog-common ---
      [INFO] 
      [INFO] --- build-helper-maven-plugin:1.5:add-source (add-source) @ tajo-catalog-common ---
      [INFO] Source directory: /home/jenkins/jenkins-slave/workspace/PreCommit-TAJO-Build/incubator-tajo/tajo-catalog/tajo-catalog-common/target/generated-sources/proto added.
      [INFO] 
      [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ tajo-catalog-common ---
      [INFO] 
      [INFO] --- maven-resources-plugin:2.2:resources (default-resources) @ tajo-catalog-common ---
      [INFO] Using default encoding to copy filtered resources.
      [INFO] 
      [INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ tajo-catalog-common ---
      [INFO] Compiling 1 source file to /home/jenkins/jenkins-slave/workspace/PreCommit-TAJO-Build/incubator-tajo/tajo-catalog/tajo-catalog-common/target/classes
      [INFO] -------------------------------------------------------------
      [ERROR] COMPILATION ERROR : 
      [INFO] -------------------------------------------------------------
      [ERROR] /home/jenkins/jenkins-slave/workspace/PreCommit-TAJO-Build/incubator-tajo/tajo-catalog/tajo-catalog-common/target/generated-sources/proto/org/apache/tajo/catalog/proto/CatalogProtos.java:[4126,55] cannot find symbol
      symbol  : class KeyValueSetProto
      location: class org.apache.tajo.rpc.protocolrecords.PrimitiveProtos
      [ERROR] /home/jenkins/jenkins-slave/workspace/PreCommit-TAJO-Build/incubator-tajo/tajo-catalog/tajo-catalog-common/target/generated-sources/proto/org/apache/tajo/catalog/proto/CatalogProtos.java:[4130,55] cannot find symbol
      symbol  : class KeyValueSetProtoOrBuilder
      

        Activity

        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-build #221 (See https://builds.apache.org/job/Tajo-master-build/221/)
        TAJO-808: Fix pre-commit build failure. (jinho) (jinossy: rev 618faa242ee0f24bc7e4b879b542b94e0f7ce357)

        • dev-support/test-patch.sh
        • CHANGES
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-build #221 (See https://builds.apache.org/job/Tajo-master-build/221/ ) TAJO-808 : Fix pre-commit build failure. (jinho) (jinossy: rev 618faa242ee0f24bc7e4b879b542b94e0f7ce357) dev-support/test-patch.sh CHANGES
        Hide
        jhkim Jinho Kim added a comment -

        Thank you for the review. I've just committed it.

        Show
        jhkim Jinho Kim added a comment - Thank you for the review. I've just committed it.
        Hide
        hyunsik Hyunsik Choi added a comment -

        +1

        It sometimes causes the failure of precommit tests. Thank you for nice finding.

        Show
        hyunsik Hyunsik Choi added a comment - +1 It sometimes causes the failure of precommit tests. Thank you for nice finding.
        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/12642602/TAJO-808.patch
        against master revision d5347c6.

        -1 @author. The patch appears to contain 1 @author tags which the Tajo community has agreed to not allow in code contributions.

        +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. 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 does not introduce any 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 .

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/407//testReport/
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/407//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/12642602/TAJO-808.patch against master revision d5347c6. -1 @author. The patch appears to contain 1 @author tags which the Tajo community has agreed to not allow in code contributions. +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. 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 does not introduce any 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 . Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/407//testReport/ Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/407//console This message is automatically generated.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development