You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@giraph.apache.org by "Avery Ching (Commented) (JIRA)" <ji...@apache.org> on 2011/10/16 08:24:11 UTC

[jira] [Commented] (GIRAPH-37) Implement Netty-backed rpc solution

    [ https://issues.apache.org/jira/browse/GIRAPH-37?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13128357#comment-13128357 ] 

Avery Ching commented on GIRAPH-37:
-----------------------------------

Jakob, this is a big patch, thanks for taking the time to try out Finagle and for the status update.  I'm hoping that Jake and/or Dmitriy will have some time and insight on dealing with some of the issues you've encountered.  In particular, the issues are a little worrisome is the reliability (2/3 of the runs only) and the compiled code with a forked version of the thrift compiler.

Note sure what this means:

https://github.com/twitter/finagle/issues/40

but it would be nice to not have to keep the generated code checked in and generated the compiled source on the fly with a maven plugin.

Also, how were the performance numbers in comparison to the current approach?


                
> Implement Netty-backed rpc solution
> -----------------------------------
>
>                 Key: GIRAPH-37
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-37
>             Project: Giraph
>          Issue Type: New Feature
>            Reporter: Jakob Homan
>            Assignee: Jakob Homan
>         Attachments: GIRAPH-37-wip.patch
>
>
> GIRAPH-12 considered replacing the current Hadoop based rpc method with Netty, but didn't went in another direction. I think there is still value in this approach, and will also look at Finagle.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira