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

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_r135516238
  
    --- Diff: 
gossip-base/src/main/java/org/apache/gossip/manager/AbstractActiveGossiper.java 
---
    @@ -78,12 +79,21 @@ public final void sendShutdownMessage(LocalMember me, 
LocalMember target){
         m.setShutdownAtNanos(gossipManager.getClock().nanoTime());
         gossipCore.sendOneWay(m, target.getUri());
       }
    -  
    -  public final void sendSharedData(LocalMember me, LocalMember member){
    -    if (member == null){
    +
    +  public final void sendSharedData(LocalMember me, LocalMember member) {
    +    if (member == null) {
           return;
         }
         long startTime = System.currentTimeMillis();
    +    if (gossipSettings.isBulkTransfer()) {
    +      sendSharedDataInBulkInternal(me, member);
    +    } else
    --- End diff --
    
    use { } even for one line if so your intention is clear.


> 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