From 8a622234759cfcb54d99c97e3c2904a1df32ca2e Mon Sep 17 00:00:00 2001
From: Viktor Baranov <baranov.viktor.27@gmail.com>
Date: Tue, 26 Oct 2021 14:48:32 +0300
Subject: [PATCH] Add config for GasUsage Cache

---
 CHANGELOG.md                                  |  1 +
 .../counters/blocks_indexed_counter.ex        |  2 +-
 apps/explorer/config/config.exs               | 11 +-----
 .../lib/explorer/chain/cache/gas_usage.ex     | 38 +++++++++++--------
 4 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3ba244e5ba..b920ea53ae 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -58,6 +58,7 @@
 - [#4582](https://github.com/blockscout/blockscout/pull/4582) - Fix NaN input on write contract page
 
 ### Chore
+- [#4819](https://github.com/blockscout/blockscout/pull/4819) - Add config for GasUsage Cache
 - [#4735](https://github.com/blockscout/blockscout/pull/4735) - Code clean up: Remove clauses for outdated ganache bugs
 - [#4726](https://github.com/blockscout/blockscout/pull/4726) - Update chart.js
 - [#4707](https://github.com/blockscout/blockscout/pull/4707) - Top navigation: Move Accounts tab to Tokens
diff --git a/apps/block_scout_web/lib/block_scout_web/counters/blocks_indexed_counter.ex b/apps/block_scout_web/lib/block_scout_web/counters/blocks_indexed_counter.ex
index 1e7d7f0746..8b8dca65fd 100644
--- a/apps/block_scout_web/lib/block_scout_web/counters/blocks_indexed_counter.ex
+++ b/apps/block_scout_web/lib/block_scout_web/counters/blocks_indexed_counter.ex
@@ -13,7 +13,7 @@ defmodule BlockScoutWeb.Counters.BlocksIndexedCounter do
   # It is undesirable to automatically start the counter in all environments.
   # Consider the test environment: if it initiates but does not finish before a
   # test ends, that test will fail.
-  config = Application.get_env(:block_scout_web, BlockScoutWeb.Counters.BlocksIndexedCounter)
+  config = Application.get_env(:block_scout_web, __MODULE__)
   @enabled Keyword.get(config, :enabled)
 
   @doc """
diff --git a/apps/explorer/config/config.exs b/apps/explorer/config/config.exs
index 5a49b27b19..493cd0e704 100644
--- a/apps/explorer/config/config.exs
+++ b/apps/explorer/config/config.exs
@@ -241,19 +241,12 @@ config :explorer, Explorer.Chain.Cache.Accounts,
   ttl_check_interval: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(1), else: false),
   global_ttl: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(5))
 
-config :explorer, Explorer.Chain.Cache.PendingTransactions,
-  enabled:
-    if(System.get_env("ETHEREUM_JSONRPC_VARIANT") == "besu",
-      do: false,
-      else: true
-    ),
-  ttl_check_interval: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(1), else: false),
-  global_ttl: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(5))
-
 config :explorer, Explorer.Chain.Cache.Uncles,
   ttl_check_interval: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(1), else: false),
   global_ttl: if(System.get_env("DISABLE_INDEXER") == "true", do: :timer.seconds(5))
 
+config :explorer, Explorer.Chain.Cache.GasUsage, enabled: false
+
 config :explorer, Explorer.ThirdPartyIntegrations.Sourcify,
   server_url: System.get_env("SOURCIFY_SERVER_URL") || "https://sourcify.dev/server",
   enabled: System.get_env("ENABLE_SOURCIFY_INTEGRATION") == "true",
diff --git a/apps/explorer/lib/explorer/chain/cache/gas_usage.ex b/apps/explorer/lib/explorer/chain/cache/gas_usage.ex
index 132f414683..ca9c29b455 100644
--- a/apps/explorer/lib/explorer/chain/cache/gas_usage.ex
+++ b/apps/explorer/lib/explorer/chain/cache/gas_usage.ex
@@ -6,6 +6,8 @@ defmodule Explorer.Chain.Cache.GasUsage do
   require Logger
 
   @default_cache_period :timer.hours(2)
+  config = Application.get_env(:explorer, __MODULE__)
+  @enabled Keyword.get(config, :enabled)
 
   use Explorer.Chain.MapCache,
     name: :gas_usage,
@@ -26,25 +28,29 @@ defmodule Explorer.Chain.Cache.GasUsage do
   end
 
   defp handle_fallback(:async_task) do
-    # If this gets called it means an async task was requested, but none exists
-    # so a new one needs to be launched
-    {:ok, task} =
-      Task.start(fn ->
-        try do
-          result = Chain.fetch_sum_gas_used()
+    if @enabled do
+      # If this gets called it means an async task was requested, but none exists
+      # so a new one needs to be launched
+      {:ok, task} =
+        Task.start(fn ->
+          try do
+            result = Chain.fetch_sum_gas_used()
 
-          set_sum(result)
-        rescue
-          e ->
-            Logger.debug([
-              "Coudn't update gas used sum test #{inspect(e)}"
-            ])
-        end
+            set_sum(result)
+          rescue
+            e ->
+              Logger.debug([
+                "Coudn't update gas used sum test #{inspect(e)}"
+              ])
+          end
 
-        set_async_task(nil)
-      end)
+          set_async_task(nil)
+        end)
 
-    {:update, task}
+      {:update, task}
+    else
+      {:update, nil}
+    end
   end
 
   # By setting this as a `callback` an async task will be started each time the