After a week of using this patch I can confirm that it works fine and
my graphs looks much better now. I hope to see it in rc4 if there are
no issues with it.

2012/9/15 Łukasz Mierzwa <[email protected]>:
> Patch works fine, I just found few carbon errors in my logs, but one
> retry was enough and there are now holes.
> I also did some testing on dev nodes with invalid carbon server ip and
> it all worked fine. For me it's tested and working.
> If You have no objections than go ahead and merge it.
> Also sending stats early after vassal reload seems like a good idea
> since I have even more gapless graphs.
>
> 2012/9/14 Łukasz Mierzwa <[email protected]>:
>> Improved patch, highlights:
>>
>> * if connection to carbon server fails we will retry (by default once,
>> after 7 seconds)
>> * after startup carbon will send first set of stats not after whole
>> cycle, but after retry delay (7 seconds), IMHO sending stats as soon
>> as possible (I'm using retry delay since it's going to be low value
>> connected somehow to frequency, at least should be)
>> * little refactoring to remove redundant code

-- 
Łukasz Mierzwa
_______________________________________________
uWSGI mailing list
[email protected]
http://lists.unbit.it/cgi-bin/mailman/listinfo/uwsgi

Reply via email to