Fixed incorrect send_after that would cause the aggregation windows t…

…o become bigger than 1 second, resulting in incorrect statistics (too high rps for example)
elli-lib · Sep 5, 2012 · e389287 · e389287
1 parent 9ebd040
commit e389287
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/elli_stats_server.erl b/src/elli_stats_server.erl
@@ -57,11 +57,11 @@ handle_cast({incr, Key, Amount}, State) ->
 
 
 handle_info(push, #state{subscribers = Subscribers} = State) ->
+    erlang:send_after(1000, self(), push),
+
     Stats = get_stats(State#state.name, State#state.elli_controller),
     Chunk = iolist_to_binary(["data: ", jiffy:encode(Stats), "\n\n"]),
     NewSubscribers = notify_subscribers(Subscribers, Chunk),
-
-    erlang:send_after(1000, self(), push),
     {noreply, State#state{subscribers = NewSubscribers}};
 
 handle_info(_Info, State) ->