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

ASF GitHub Bot commented on GOSSIP-41:
--------------------------------------

Github user edwardcapriolo commented on a diff in the pull request:

    https://github.com/apache/incubator-gossip/pull/68#discussion_r134644785
  
    --- Diff: 
gossip-base/src/main/java/org/apache/gossip/manager/handlers/MessageHandlerFactory.java
 ---
    @@ -35,8 +30,8 @@ public static MessageHandler defaultHandler() {
         return concurrentHandler(
             new TypedMessageHandler(Response.class, new ResponseHandler()),
             new TypedMessageHandler(ShutdownMessage.class, new 
ShutdownMessageHandler()),
    -        new TypedMessageHandler(PerNodeDataMessage.class, new 
PerNodeDataMessageHandler()),
    --- End diff --
    
    Lets not remove the old system. Eventually I think the bulk only way makes 
sense but there is no issue in supporting both in the code and choosing with a 
switch


> Transfer gossip data in bulk
> ----------------------------
>
>                 Key: GOSSIP-41
>                 URL: https://issues.apache.org/jira/browse/GOSSIP-41
>             Project: Gossip
>          Issue Type: Improvement
>            Reporter: Edward Capriolo
>            Assignee: Pooya Salehi
>
> Currently the process that transfers shared data and per node data iterates a 
> map transmitting a single datum at a time using one way communication.  We 
> could transmit more key values at once by simply saying if keys < 100 
> transfer all at once, or by collecting keys and sending in groups of 100. 
> This would greatly lower chatter communicating data.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to