Details

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

      Description

      Seemed remained by mistake. Removing one in rpc-common package.

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #623 (See https://builds.apache.org/job/Tajo-master-build/623/)
        TAJO-1414: Two RemoteException in rpc module (jihun: rev 5260c1e064e09f14b686144bdfc136fe76e01f4d)

        • tajo-rpc/tajo-rpc-protobuf/src/main/java/org/apache/tajo/rpc/RemoteException.java
        • CHANGES
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #623 (See https://builds.apache.org/job/Tajo-master-build/623/ ) TAJO-1414 : Two RemoteException in rpc module (jihun: rev 5260c1e064e09f14b686144bdfc136fe76e01f4d) tajo-rpc/tajo-rpc-protobuf/src/main/java/org/apache/tajo/rpc/RemoteException.java CHANGES
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #260 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/260/)
        TAJO-1414: Two RemoteException in rpc module (jihun: rev 5260c1e064e09f14b686144bdfc136fe76e01f4d)

        • tajo-rpc/tajo-rpc-protobuf/src/main/java/org/apache/tajo/rpc/RemoteException.java
        • CHANGES
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #260 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/260/ ) TAJO-1414 : Two RemoteException in rpc module (jihun: rev 5260c1e064e09f14b686144bdfc136fe76e01f4d) tajo-rpc/tajo-rpc-protobuf/src/main/java/org/apache/tajo/rpc/RemoteException.java CHANGES
        Hide
        ykrips Jihun Kang added a comment -

        Committed.

        Show
        ykrips Jihun Kang added a comment - Committed.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/435

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/435
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ykrips commented on the pull request:

        https://github.com/apache/tajo/pull/435#issuecomment-83867201

        +1.
        Though travis ci buils is still in progress, but Maven build has succeeded on my local machine.
        ```
        [INFO] Reactor Summary:
        [INFO]
        [INFO] Tajo Main .......................................... SUCCESS [ 1.726 s]
        [INFO] Tajo Project POM ................................... SUCCESS [ 0.975 s]
        [INFO] Tajo Maven Plugins ................................. SUCCESS [ 3.163 s]
        [INFO] Tajo Common ........................................ SUCCESS [ 18.845 s]
        [INFO] Tajo Algebra ....................................... SUCCESS [ 1.381 s]
        [INFO] Tajo Catalog Common ................................ SUCCESS [ 6.270 s]
        [INFO] Tajo Plan .......................................... SUCCESS [ 5.521 s]
        [INFO] Tajo Rpc Common .................................... SUCCESS [ 0.448 s]
        [INFO] Tajo Protocol Buffer Rpc ........................... SUCCESS [ 20.159 s]
        [INFO] Tajo Catalog Client ................................ SUCCESS [ 1.596 s]
        [INFO] Tajo Catalog Server ................................ SUCCESS [01:19 min]
        [INFO] Tajo Storage Common ................................ SUCCESS [ 9.753 s]
        [INFO] Tajo HDFS Storage .................................. SUCCESS [ 42.392 s]
        [INFO] Tajo HBase Storage ................................. SUCCESS [ 3.377 s]
        [INFO] Tajo PullServer .................................... SUCCESS [ 1.364 s]
        [INFO] Tajo Client ........................................ SUCCESS [ 2.590 s]
        [INFO] Tajo CLI tools ..................................... SUCCESS [ 0.919 s]
        [INFO] Tajo JDBC Driver ................................... SUCCESS [ 2.320 s]
        [INFO] ASM (thirdparty) ................................... SUCCESS [ 0.531 s]
        [INFO] Tajo Catalog Drivers HCatalog ...................... SUCCESS [ 30.270 s]
        [INFO] Tajo Core .......................................... SUCCESS [11:57 min]
        [INFO] Tajo RESTful Container ............................. SUCCESS [ 3.161 s]
        [INFO] Tajo RPC ........................................... SUCCESS [ 0.041 s]
        [INFO] Tajo Catalog Drivers ............................... SUCCESS [ 0.042 s]
        [INFO] Tajo Catalog ....................................... SUCCESS [ 0.106 s]
        [INFO] Tajo Storage ....................................... SUCCESS [ 0.057 s]
        [INFO] Tajo Distribution .................................. SUCCESS [ 0.639 s]
        [INFO] ------------------------------------------------------------------------
        [INFO] BUILD SUCCESS
        [INFO] ------------------------------------------------------------------------
        [INFO] Total time: 15:55 min
        [INFO] Finished at: 2015-03-20T10:27:09+09:00
        [INFO] Final Memory: 92M/749M
        [INFO] ------------------------------------------------------------------------
        ```

        Show
        githubbot ASF GitHub Bot added a comment - Github user ykrips commented on the pull request: https://github.com/apache/tajo/pull/435#issuecomment-83867201 +1. Though travis ci buils is still in progress, but Maven build has succeeded on my local machine. ``` [INFO] Reactor Summary: [INFO] [INFO] Tajo Main .......................................... SUCCESS [ 1.726 s] [INFO] Tajo Project POM ................................... SUCCESS [ 0.975 s] [INFO] Tajo Maven Plugins ................................. SUCCESS [ 3.163 s] [INFO] Tajo Common ........................................ SUCCESS [ 18.845 s] [INFO] Tajo Algebra ....................................... SUCCESS [ 1.381 s] [INFO] Tajo Catalog Common ................................ SUCCESS [ 6.270 s] [INFO] Tajo Plan .......................................... SUCCESS [ 5.521 s] [INFO] Tajo Rpc Common .................................... SUCCESS [ 0.448 s] [INFO] Tajo Protocol Buffer Rpc ........................... SUCCESS [ 20.159 s] [INFO] Tajo Catalog Client ................................ SUCCESS [ 1.596 s] [INFO] Tajo Catalog Server ................................ SUCCESS [01:19 min] [INFO] Tajo Storage Common ................................ SUCCESS [ 9.753 s] [INFO] Tajo HDFS Storage .................................. SUCCESS [ 42.392 s] [INFO] Tajo HBase Storage ................................. SUCCESS [ 3.377 s] [INFO] Tajo PullServer .................................... SUCCESS [ 1.364 s] [INFO] Tajo Client ........................................ SUCCESS [ 2.590 s] [INFO] Tajo CLI tools ..................................... SUCCESS [ 0.919 s] [INFO] Tajo JDBC Driver ................................... SUCCESS [ 2.320 s] [INFO] ASM (thirdparty) ................................... SUCCESS [ 0.531 s] [INFO] Tajo Catalog Drivers HCatalog ...................... SUCCESS [ 30.270 s] [INFO] Tajo Core .......................................... SUCCESS [11:57 min] [INFO] Tajo RESTful Container ............................. SUCCESS [ 3.161 s] [INFO] Tajo RPC ........................................... SUCCESS [ 0.041 s] [INFO] Tajo Catalog Drivers ............................... SUCCESS [ 0.042 s] [INFO] Tajo Catalog ....................................... SUCCESS [ 0.106 s] [INFO] Tajo Storage ....................................... SUCCESS [ 0.057 s] [INFO] Tajo Distribution .................................. SUCCESS [ 0.639 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 15:55 min [INFO] Finished at: 2015-03-20T10:27:09+09:00 [INFO] Final Memory: 92M/749M [INFO] ------------------------------------------------------------------------ ```
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user navis commented on the pull request:

        https://github.com/apache/tajo/pull/435#issuecomment-83252404

        ok, sure.

        Show
        githubbot ASF GitHub Bot added a comment - Github user navis commented on the pull request: https://github.com/apache/tajo/pull/435#issuecomment-83252404 ok, sure.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ykrips commented on the pull request:

        https://github.com/apache/tajo/pull/435#issuecomment-83247819

        Can you remove exception in tajo-rpc-protobuf module? This exception is common class and it could be used in rest module in future.

        Show
        githubbot ASF GitHub Bot added a comment - Github user ykrips commented on the pull request: https://github.com/apache/tajo/pull/435#issuecomment-83247819 Can you remove exception in tajo-rpc-protobuf module? This exception is common class and it could be used in rest module in future.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ykrips commented on the pull request:

        https://github.com/apache/tajo/pull/435#issuecomment-83231501

        My bad. When travis ci build has finished, this patch will be merged into main branch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user ykrips commented on the pull request: https://github.com/apache/tajo/pull/435#issuecomment-83231501 My bad. When travis ci build has finished, this patch will be merged into main branch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user navis opened a pull request:

        https://github.com/apache/tajo/pull/435

        TAJO-1414 Two RemoteException in rpc module

        trivial

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/navis/tajo TAJO-1414

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/435.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #435


        commit 9f1ec4e175a180dffe08508cf8a82169dfec44fd
        Author: navis.ryu <navis@apache.org>
        Date: 2015-03-18T23:59:46Z

        TAJO-1414 Two RemoteException in rpc module


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user navis opened a pull request: https://github.com/apache/tajo/pull/435 TAJO-1414 Two RemoteException in rpc module trivial You can merge this pull request into a Git repository by running: $ git pull https://github.com/navis/tajo TAJO-1414 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/435.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #435 commit 9f1ec4e175a180dffe08508cf8a82169dfec44fd Author: navis.ryu <navis@apache.org> Date: 2015-03-18T23:59:46Z TAJO-1414 Two RemoteException in rpc module

          People

          • Assignee:
            navis Navis
            Reporter:
            navis Navis
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development