Skip to content

Commit

Permalink
fix: consolidate address w/ balance one at a time
Browse files Browse the repository at this point in the history
  • Loading branch information
zachdaniel committed May 20, 2019
1 parent 587d7f4 commit 0a32431
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 10 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
- [#1956](https://github.com/poanetwork/blockscout/pull/1956) - add logs tab to address
- [#1933](https://github.com/poanetwork/blockscout/pull/1933) - add eth_BlockNumber json rpc method
- [#1952](https://github.com/poanetwork/blockscout/pull/1952) - feat: exclude empty contracts by default
- [#1989](https://github.com/poanetwork/blockscout/pull/1989) - fix: consolidate address w/ balance one at a time

### Fixes

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,15 +40,10 @@ defmodule Explorer.Counters.AddressesWithBalanceCounter do
end

@impl true
def init(args) do
def init(_args) do
create_table()

if enable_consolidation?() do
Task.start_link(&consolidate/0)
schedule_next_consolidation()
end

{:ok, args}
{:ok, %{consolidate?: enable_consolidation?()}, {:continue, :ok}}
end

def create_table do
Expand All @@ -63,9 +58,7 @@ defmodule Explorer.Counters.AddressesWithBalanceCounter do
end

defp schedule_next_consolidation do
if enable_consolidation?() do
Process.send_after(self(), :consolidate, :timer.seconds(@update_interval_in_seconds))
end
Process.send_after(self(), :consolidate, :timer.seconds(@update_interval_in_seconds))
end

@doc """
Expand All @@ -75,6 +68,19 @@ defmodule Explorer.Counters.AddressesWithBalanceCounter do
:ets.insert(table_name(), {key, info})
end

@impl true
def handle_continue(:ok, %{consolidate?: true} = state) do
consolidate()
schedule_next_consolidation()

{:noreply, state}
end

@impl true
def handle_continue(:ok, state) do
{:noreply, state}
end

@impl true
def handle_info(:consolidate, state) do
consolidate()
Expand Down

0 comments on commit 0a32431

Please sign in to comment.