You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by ko...@apache.org on 2009/06/30 21:15:54 UTC

svn commit: r789884 - /couchdb/trunk/src/couchdb/couch_stats_aggregator.erl

Author: kocolosk
Date: Tue Jun 30 19:15:54 2009
New Revision: 789884

URL: http://svn.apache.org/viewvc?rev=789884&view=rev
Log:
correct stat intervals.  Thanks Bob Dionne

Modified:
    couchdb/trunk/src/couchdb/couch_stats_aggregator.erl

Modified: couchdb/trunk/src/couchdb/couch_stats_aggregator.erl
URL: http://svn.apache.org/viewvc/couchdb/trunk/src/couchdb/couch_stats_aggregator.erl?rev=789884&r1=789883&r2=789884&view=diff
==============================================================================
--- couchdb/trunk/src/couchdb/couch_stats_aggregator.erl (original)
+++ couchdb/trunk/src/couchdb/couch_stats_aggregator.erl Tue Jun 30 19:15:54 2009
@@ -355,9 +355,9 @@
     % fifteen minutes) are gone.
 
     {ok, _} = timer:apply_interval(1000, ?MODULE, time_passed, []),
-    {ok, _} = timer:apply_interval(6000, ?MODULE, clear_aggregates, ['60']),
-    {ok, _} = timer:apply_interval(30000, ?MODULE, clear_aggregates, ['300']),
-    {ok, _} = timer:apply_interval(90000, ?MODULE, clear_aggregates, ['900']).
+    {ok, _} = timer:apply_interval(60000, ?MODULE, clear_aggregates, ['60']),
+    {ok, _} = timer:apply_interval(300000, ?MODULE, clear_aggregates, ['300']),
+    {ok, _} = timer:apply_interval(900000, ?MODULE, clear_aggregates, ['900']).
 
 
 % Unused gen_server behaviour API functions that we need to declare.



Re: svn commit: r789884 - /couchdb/trunk/src/couchdb/couch_stats_aggregator.erl

Posted by Noah Slater <ns...@apache.org>.
On Thu, Jul 02, 2009 at 09:17:57AM -0400, Adam Kocoloski wrote:
> Hi Jan, I hadn't planned on merging it, but I'm not opposed to someone
> doing so.  Cheers,

And this is why I waited a few days before doing another vote. Heh.

Let me know when you're done guys!

Thanks,

-- 
Noah Slater, http://tumbolia.org/nslater

Re: svn commit: r789884 - /couchdb/trunk/src/couchdb/couch_stats_aggregator.erl

Posted by Adam Kocoloski <ko...@apache.org>.
On Jul 1, 2009, at 8:20 AM, Jan Lehnardt wrote:

>
> On 30 Jun 2009, at 21:15, kocolosk@apache.org wrote:
>
>> Author: kocolosk
>> Date: Tue Jun 30 19:15:54 2009
>> New Revision: 789884
>>
>> URL: http://svn.apache.org/viewvc?rev=789884&view=rev
>> Log:
>> correct stat intervals.  Thanks Bob Dionne
>
> Good catch. Do we want to merge that into 0.9.1?

Hi Jan, I hadn't planned on merging it, but I'm not opposed to someone  
doing so.  Cheers,

Adam

Re: svn commit: r789884 - /couchdb/trunk/src/couchdb/couch_stats_aggregator.erl

Posted by Jan Lehnardt <ja...@apache.org>.
On 30 Jun 2009, at 21:15, kocolosk@apache.org wrote:

> Author: kocolosk
> Date: Tue Jun 30 19:15:54 2009
> New Revision: 789884
>
> URL: http://svn.apache.org/viewvc?rev=789884&view=rev
> Log:
> correct stat intervals.  Thanks Bob Dionne

Good catch. Do we want to merge that into 0.9.1?

Cheers
Jan
--


>
> Modified:
>    couchdb/trunk/src/couchdb/couch_stats_aggregator.erl
>
> Modified: couchdb/trunk/src/couchdb/couch_stats_aggregator.erl
> URL: http://svn.apache.org/viewvc/couchdb/trunk/src/couchdb/couch_stats_aggregator.erl?rev=789884&r1=789883&r2=789884&view=diff
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- couchdb/trunk/src/couchdb/couch_stats_aggregator.erl (original)
> +++ couchdb/trunk/src/couchdb/couch_stats_aggregator.erl Tue Jun 30  
> 19:15:54 2009
> @@ -355,9 +355,9 @@
>     % fifteen minutes) are gone.
>
>     {ok, _} = timer:apply_interval(1000, ?MODULE, time_passed, []),
> -    {ok, _} = timer:apply_interval(6000, ?MODULE, clear_aggregates,  
> ['60']),
> -    {ok, _} = timer:apply_interval(30000, ?MODULE,  
> clear_aggregates, ['300']),
> -    {ok, _} = timer:apply_interval(90000, ?MODULE,  
> clear_aggregates, ['900']).
> +    {ok, _} = timer:apply_interval(60000, ?MODULE,  
> clear_aggregates, ['60']),
> +    {ok, _} = timer:apply_interval(300000, ?MODULE,  
> clear_aggregates, ['300']),
> +    {ok, _} = timer:apply_interval(900000, ?MODULE,  
> clear_aggregates, ['900']).
>
>
> % Unused gen_server behaviour API functions that we need to declare.
>
>