[jira] [Commented] (GOSSIP-35) Different payloads can be transmitted at different rates

2017-08-03 Thread Juby Victor (JIRA)

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

Juby Victor commented on GOSSIP-35:
---

Introducing 5 additional parameters into the class will either need an 
overloaded constructor or change the existing constructor. IMHO a constructor 
with 10 parameters is really clumsy, So we have the option of introducing a 
builder, or stick with a 5 param constructor & 10 param constructor. I'm 
inclined to option 1 (Builder). Since there are  only two places which uses the 
settings class, I guess we can bite the bullet on this and make the change to 
use a builder - this should give us some flexibility to introduce other params 
further down the line too...

> Different payloads can be transmitted at different rates
> 
>
> Key: GOSSIP-35
> URL: https://issues.apache.org/jira/browse/GOSSIP-35
> Project: Gossip
>  Issue Type: New Feature
>Reporter: Edward Capriolo
>Assignee: Juby Victor
> Fix For: 0.1.3
>
>
> Current we have a a few ScheduledExectuor one that controls active gossip, 
> one that controls gossip to dead nodes, one that controls gosssip of user 
> data, etc. Currently they all use the same setting to control how often they 
> run (10ms currently). It would be better if they each had their own setting. 



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


[jira] [Commented] (GOSSIP-35) Different payloads can be transmitted at different rates

2017-08-02 Thread Edward Capriolo (JIRA)

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

Edward Capriolo commented on GOSSIP-35:
---

Good analysis.
{quote}
Should we support overriding these properties via environment variables ?
{quote}
No, I do not think we  support any configuration through environment variables. 
There is GossipSettings and optionally the json file support adding a third way 
to hydrate variables is too confusing.

{quote}
side effects on gossip-base/SimpleActiveGossipper#init 
{quote}
There is a second instance of active Gossiper that is network aware. You should 
consider what they does as well and how it would respond to these variables.  

> Different payloads can be transmitted at different rates
> 
>
> Key: GOSSIP-35
> URL: https://issues.apache.org/jira/browse/GOSSIP-35
> Project: Gossip
>  Issue Type: New Feature
>Reporter: Edward Capriolo
>Assignee: Juby Victor
> Fix For: 0.1.3
>
>
> Current we have a a few ScheduledExectuor one that controls active gossip, 
> one that controls gossip to dead nodes, one that controls gosssip of user 
> data, etc. Currently they all use the same setting to control how often they 
> run (10ms currently). It would be better if they each had their own setting. 



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


[jira] [Commented] (GOSSIP-35) Different payloads can be transmitted at different rates

2017-07-31 Thread Edward Capriolo (JIRA)

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

Edward Capriolo commented on GOSSIP-35:
---

I assigned this issue to you [~jubyvictor]. Let me know how I can help if need 
be,.

> Different payloads can be transmitted at different rates
> 
>
> Key: GOSSIP-35
> URL: https://issues.apache.org/jira/browse/GOSSIP-35
> Project: Gossip
>  Issue Type: New Feature
>Reporter: Edward Capriolo
>Assignee: Juby Victor
> Fix For: 0.1.3
>
>
> Current we have a a few ScheduledExectuor one that controls active gossip, 
> one that controls gossip to dead nodes, one that controls gosssip of user 
> data, etc. Currently they all use the same setting to control how often they 
> run (10ms currently). It would be better if they each had their own setting. 



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


[jira] [Commented] (GOSSIP-35) Different payloads can be transmitted at different rates

2017-07-30 Thread Juby Victor (JIRA)

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

Juby Victor commented on GOSSIP-35:
---

Hi Edward I would like to work on this issue. 


> Different payloads can be transmitted at different rates
> 
>
> Key: GOSSIP-35
> URL: https://issues.apache.org/jira/browse/GOSSIP-35
> Project: Gossip
>  Issue Type: New Feature
>Reporter: Edward Capriolo
> Fix For: 0.1.3
>
>
> Current we have a a few ScheduledExectuor one that controls active gossip, 
> one that controls gossip to dead nodes, one that controls gosssip of user 
> data, etc. Currently they all use the same setting to control how often they 
> run (10ms currently). It would be better if they each had their own setting. 



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