pghero 1.5.3 → 2.0.0

data/app/views/pg_hero/home/_live_queries_table.html.erb

@@ -8,24 +8,27 @@
     </tr>
   </thead>
   <tbody>
-    <% now = Time.now %>
     <% queries.reverse.each do |query| %>
       <tr>
-        <td><%= query["pid"] %></td>
-        <td><%= query["started_at"] ? "#{number_with_delimiter(((now - Time.parse(query["started_at"])) * 1000).round)} ms" : nil %></td>
-        <td><%= query["state"] %></td>
+        <td><%= query[:pid] %></td>
+        <td><%= number_with_delimiter(query[:duration_ms].round) %> ms</td>
+        <td><%= query[:state] %></td>
         <td class="text-right">
-          <% button_path, button_options = Rails.version >= "4.1" ? [explain_path, {params: {query: query["query"]}}] : [explain_path(query: query["query"]), {}] %>
+          <% button_path, button_options = Rails.version >= "4.1" ? [explain_path, {params: {query: query[:query]}}] : [explain_path(query: query[:query]), {}] %>
           <%= button_to "Explain", button_path, button_options.merge(form: {target: "_blank"}, class: "btn btn-info") %>
-          <%= button_to "Kill", kill_path(pid: query["pid"]), class: "btn btn-danger" %>
+          <%= button_to "Kill", kill_path(pid: query[:pid]), class: "btn btn-danger" %>
         </td>
       </tr>
       <tr>
         <td colspan="6" style="border-top: none; padding: 0;">
-          <%= query["source"] %> <span class="text-muted"><%= query["user"] %></span>
-          <pre><%= query["query"] %></pre>
+          <%= query[:source] %> <span class="text-muted"><%= query[:user] %></span>
+          <pre style="margin-top: 1em;"><code><%= query[:query] %></code></pre>
         </td>
       </tr>
     <% end %>
   </tbody>
 </table>
+
+<script>
+  highlightQueries();
+</script>

data/app/views/pg_hero/home/_queries_table.html.erb

@@ -25,9 +25,9 @@
     <% queries.each do |query| %>
       <tr>
         <td>
-          <%= number_with_delimiter(query["total_minutes"].to_f.round) %> min
+          <%= number_with_delimiter(query[:total_minutes].round) %> min
           <span class="percent">
-            <% percent = query["total_percent"].to_f %>
+            <% percent = query[:total_percent] %>
             <% if percent > 1 %>
               <%= percent.round %>%
             <% elsif percent > 0.1 %>
@@ -37,21 +37,32 @@
             <% end %>
           </span>
         </td>
-        <td><%= number_with_delimiter(query["average_time"].to_f.round) %> ms</td>
         <td>
-          <%= number_with_delimiter(query["calls"].to_i) %>
-          <% if query["user"] %>
-            <span class="user"><%= query["user"] %></span>
-          <% end %>
+          <%= number_with_delimiter(query[:average_time].round) %> ms
+        </td>
+        <td>
+          <%= number_with_delimiter(query[:calls]) %>
+
+          <span class="user">
+            <% if query[:user] %>
+              <%= query[:user] %>
+              <% if @show_details %>
+                &middot;
+              <% end %>
+            <% end %>
+            <% if @show_details %>
+              <%= link_to "details", show_query_path(query[:query_hash]), target: "_blank" %>
+            <% end %>
+          </span>
         </td>
       </tr>
       <tr>
         <td colspan="3" style="border-top: none; padding: 0;">
-          <code><pre style="max-height: 230px; overflow: hidden;" onclick="this.style.maxHeight = 'none';"><%= query["query"] %></pre></code>
-          <% if query["query"] == "<insufficient privilege>" %>
+          <pre><code style="max-height: 230px; overflow: hidden;" onclick="this.style.maxHeight = 'none';"><%= query[:query] %></code></pre>
+          <% if query[:query] == "<insufficient privilege>" %>
             <p class="text-muted">For security reasons, only superusers can see queries executed by other users.</p>
           <% end %>
-          <% if local_assigns[:suggested_indexes] != false && (details = @suggested_indexes_by_query[query["query"]]) %>
+          <% if local_assigns[:suggested_indexes] != false && (details = @suggested_indexes_by_query[query[:query]]) && (details[:index] || @debug) %>
             <%= render partial: "suggested_index", locals: {index: details[:index], details: details} %>
           <% end %>
         </td>

data/app/views/pg_hero/home/_suggested_index.html.erb

@@ -13,6 +13,6 @@ Row estimates
 <%= details[:row_estimates].to_a.map { |k, v| "- #{k}: #{v}" }.join("\n") %><% end %><% if details[:table_indexes] %>
 
 Existing indexes
-<% details[:table_indexes].sort_by { |i| [PgHero.truthy?(i["primary"]) ? 0 : 1, i["columns"]] }.each do |i3| %>- <%= i3["columns"].join(", ") %><% if i3["using"] != "btree" %> <%= i3["using"].to_s.upcase %><% end %><% if PgHero.truthy?(i3["primary"]) %> PRIMARY<% elsif PgHero.truthy?(i3["unique"]) %> UNIQUE<% end %>
+<% details[:table_indexes].sort_by { |i| [i[:primary] ? 0 : 1, i[:columns]] }.each do |i3| %>- <%= i3[:columns].join(", ") %><% if i3[:using] != "btree" %> <%= i3[:using].to_s.upcase %><% end %><% if i3[:primary] %> PRIMARY<% elsif i3[:unique] %> UNIQUE<% end %>
 <% end %><% end %></pre></code>
 </div>

data/app/views/pg_hero/home/connections.html.erb

@@ -1,5 +1,23 @@
 <div class="content">
   <h1>Connections</h1>
 
-  <%= render partial: "connections_table", locals: {total_connections: @total_connections, connection_sources: @connection_sources, show_message: false} %>
+  <p><%= pluralize(@total_connections, "connection") %></p>
+
+  <% if @total_connections > 0 %>
+    <h3>By Database</h3>
+
+    <div id="chart-1" class="chart" style="height: 260px; line-height: 260px; margin-bottom: 20px;">Loading...</div>
+    <script>
+      new Chartkick.PieChart("chart-1", <%= json_escape(@connections_by_database.to_json).html_safe %>);
+    </script>
+
+    <h3>By User</h3>
+
+    <div id="chart-2" class="chart" style="height: 260px; line-height: 260px; margin-bottom: 20px;">Loading...</div>
+    <script>
+      new Chartkick.PieChart("chart-2", <%= json_escape(@connections_by_user.to_json).html_safe %>);
+    </script>
+
+    <%= render partial: "connections_table", locals: {connection_sources: @connection_sources} %>
+  <% end %>
 </div>

data/app/views/pg_hero/home/explain.html.erb

@@ -14,7 +14,7 @@
     <% if @visualize %>
       <p>Paste the output below into the <%= link_to "Postgres Explain Visualizer", "http://tatiyants.com/pev/#/plans/new", target: "_blank" %></p>
     <% end %>
-    <pre><%= @explanation %></pre>
+    <pre><code><%= @explanation %></code></pre>
     <% unless @visualize %>
       <p><%= link_to "See how to interpret this", "http://www.postgresql.org/docs/current/static/using-explain.html", target: "_blank" %></p>
     <% end %>

data/app/views/pg_hero/home/index.html.erb

@@ -8,6 +8,10 @@
       <% end %>
       <span class="tiny"><%= number_with_delimiter((@replication_lag * 1000).round) %> ms</span>
     </div>
+  <% elsif @inactive_replication_slots.any? %>
+    <div class="alert alert-warning">
+      <%= pluralize(@inactive_replication_slots.size, "inactive replication slot") %>
+    </div>
   <% end %>
   <div class="alert alert-<%= @long_running_queries.empty? ? "success" : "warning" %>">
     <% if @long_running_queries.any? %>
@@ -15,17 +19,22 @@
     <% else %>
       No long running queries
     <% end %>
-  </div>
-  <div class="alert alert-<%= @good_cache_rate ? "success" : "warning" %>">
-    <% if @good_cache_rate %>
-      Cache hit rate above <%= @database.cache_hit_rate_threshold %>%
-    <% else %>
-      Low cache hit rate
+    <% if @autovacuum_queries.any? %>
+      <span class="tiny"><%= @autovacuum_queries.size %> autovacuum</span>
     <% end %>
   </div>
+  <% if @extended %>
+    <div class="alert alert-<%= @good_cache_rate ? "success" : "warning" %>">
+      <% if @good_cache_rate %>
+        Cache hit rate above <%= @database.cache_hit_rate_threshold %>%
+      <% else %>
+        Low cache hit rate
+      <% end %>
+    </div>
+  <% end %>
   <div class="alert alert-<%= @good_total_connections ? "success" : "warning" %>">
     <% if @good_total_connections %>
-      Healthy number of connections
+      Number of connections healthy
     <% else %>
       High number of connections
     <% end %>
@@ -42,7 +51,7 @@
     <% if @sequence_danger.any? %>
       <%= pluralize(@sequence_danger.size, "columns") %> approaching overflow
     <% else %>
-      No columns near overflow
+      No columns near integer overflow
     <% end %>
   </div>
   <div class="alert alert-<%= @invalid_indexes.empty? ? "success" : "warning" %>">
@@ -69,14 +78,6 @@
         No suggested indexes
       <% end %>
     </div>
-  <% else %>
-    <div class="alert alert-<%= @missing_indexes.empty? ? "success" : "warning" %>">
-      <% if @missing_indexes.any? %>
-        <%= pluralize(@missing_indexes.size, "table appears", "tables appear") %> to be missing indexes
-      <% else %>
-        No missing indexes
-      <% end %>
-    </div>
   <% end %>
   <div class="alert alert-<%= @query_stats_enabled && @slow_queries.empty? ? "success" : "warning" %>">
     <% if !@query_stats_enabled %>
@@ -87,6 +88,15 @@
       No slow queries
     <% end %>
   </div>
+  <% if @extended %>
+    <div class="alert alert-<%= @unused_indexes.empty? ? "success" : "warning" %>">
+      <% if @unused_indexes.any? %>
+        <%= pluralize(@unused_indexes.size, "unused index", "unused indexes") %>
+      <% else %>
+        No unused indexes
+      <% end %>
+    </div>
+  <% end %>
 </div>
 
 <% if @replica && !@good_replication_lag %>
@@ -97,6 +107,29 @@
   </div>
 <% end %>
 
+<% if @inactive_replication_slots && @inactive_replication_slots.any? %>
+  <div class="content">
+    <h1>Inactive Replication Slots</h1>
+    <p>Inactive replication slots can cause a lot of disk space to be consumed.</p>
+    <p>For each, run:</p>
+    <pre><code>SELECT pg_drop_replication_slot('slot_name');</code></pre>
+    <table class="table">
+      <thead>
+        <tr>
+          <th>Name</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% @inactive_replication_slots.each do |slot| %>
+          <tr>
+            <td><%= slot[:slot_name] %></td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table>
+  </div>
+<% end %>
+
 <% if @long_running_queries.any? %>
   <div class="content">
     <%= button_to "Kill All", kill_long_running_queries_path, class: "btn btn-danger", style: "float: right;" %>
@@ -106,7 +139,7 @@
   </div>
 <% end %>
 
-<% if !@good_cache_rate %>
+<% if @extended && !@good_cache_rate %>
   <div class="content">
     <h1>Low Cache Hit Rate</h1>
 
@@ -126,7 +159,11 @@
 <% if !@good_total_connections %>
   <div class="content">
     <h1>High Number of Connections</h1>
-    <%= render partial: "connections_table", locals: {total_connections: @total_connections, connection_sources: @database.connection_sources(by_database_and_user: true).first(10), show_message: true} %>
+    <p><%= pluralize(@total_connections, "connection") %></p>
+
+    <p><%= link_to "Use connection pooling", "http://www.craigkerstiens.com/2014/05/22/on-connection-pooling/", target: "_blank" %> for better performance. <%= link_to "PgBouncer", "https://wiki.postgresql.org/wiki/PgBouncer", target: "_blank" %> is a solid option.</p>
+
+    <%= render partial: "connections_table", locals: {connection_sources: @database.connection_sources.first(10)} %>
   </div>
 <% end %>
 
@@ -135,7 +172,7 @@
     <h2>Vacuuming Needed</h2>
     <p>The database <strong>will shutdown</strong> when there are fewer than 1,000,000 transactions left. <%= link_to "Read more", "http://www.postgresql.org/docs/9.1/static/routine-vacuuming.html#VACUUM-FOR-WRAPAROUND", target: "_blank" %>.</p>
     <p>For each table, run:</p>
-    <code><pre>VACUUM FREEZE VERBOSE table</pre></code>
+    <pre><code>VACUUM FREEZE VERBOSE table;</code></pre>
     <table class="table">
       <thead>
         <tr>
@@ -146,8 +183,13 @@
       <tbody>
         <% @transaction_id_danger.each do |query| %>
           <tr>
-            <td><%= query["table"] %></td>
-            <td><%= number_with_delimiter(query["transactions_before_shutdown"]) %></td>
+            <td>
+              <%= query[:table] %>
+              <% if query[:schema] != "public" %>
+                <span class="text-muted"><%= query[:schema] %></span>
+              <% end %>
+            </td>
+            <td><%= number_with_delimiter(query[:transactions_left]) %></td>
           </tr>
         <% end %>
       </tbody>
@@ -169,19 +211,19 @@
         </tr>
       </thead>
       <tbody>
-        <% @sequence_danger.sort_by { |s| [s["table"], s["column"]] }.each do |query| %>
+        <% @sequence_danger.sort_by { |s| [s[:table], s[:column]] }.each do |query| %>
           <tr>
             <td>
-              <%= query["table"] %>.<%= query["column"] %>
+              <%= query[:table] %>.<%= query[:column] %>
             </td>
             <td>
-              <%= query["column_type"] %>
+              <%= query[:column_type] %>
             </td>
             <td>
-              <%= number_with_delimiter(query["max_value"].to_i - query["last_value"].to_i) %>
+              <%= number_with_delimiter(query[:max_value] - query[:last_value]) %>
             </td>
             <td>
-              <%= number_to_percentage((query["max_value"].to_i - query["last_value"].to_i) * 100.0 / query["max_value"].to_i, precision: 2, significant: true) %>
+              <%= number_to_percentage((query[:max_value] - query[:last_value]) * 100.0 / query[:max_value], precision: 2, significant: true) %>
             </td>
           </tr>
         <% end %>
@@ -203,9 +245,9 @@
         </tr>
       </thead>
       <tbody>
-        <% @invalid_indexes.each do |query| %>
+        <% @invalid_indexes.each do |index| %>
           <tr>
-            <td><%= query["index"] %></td>
+            <td><%= index[:name] %></td>
           </tr>
         <% end %>
       </tbody>
@@ -229,10 +271,10 @@
       <pre>rails g migration remove_unneeded_indexes</pre>
       <p>And paste</p>
       <pre style="overflow: scroll; white-space: pre; word-break: normal;"><% @duplicate_indexes.each do |query| %>
-remove_index <%= query["unneeded_index"]["table"].to_sym.inspect %>, name: <%= query["unneeded_index"]["name"].to_s.inspect %><% end %></pre>
+remove_index <%= query[:unneeded_index][:table].to_sym.inspect %>, name: <%= query[:unneeded_index][:name].to_s.inspect %><% end %></pre>
     </div>
 
-    <table class="table">
+    <table class="table duplicate-indexes">
       <thead>
         <tr>
           <th>Details</th>
@@ -240,14 +282,14 @@ remove_index <%= query["unneeded_index"]["table"].to_sym.inspect %>, name: <%= q
       </thead>
       <tbody>
         <% @duplicate_indexes.each do |index| %>
-          <% unneeded_index = index["unneeded_index"] %>
-          <% covering_index = index["covering_index"] %>
+          <% unneeded_index = index[:unneeded_index] %>
+          <% covering_index = index[:covering_index] %>
           <tr>
-            <td style="padding-top: 15px; padding-bottom: 5px;">
-              On <%= unneeded_index["table"] %>
-              <pre><%= unneeded_index["name"] %> (<%= unneeded_index["columns"].join(", ") %>)</pre>
+            <td>
+              On <%= unneeded_index[:table] %>
+              <pre><%= unneeded_index[:name] %> (<%= unneeded_index[:columns].join(", ") %>)</pre>
               is covered by
-              <pre><%= covering_index["name"] %> (<%= covering_index["columns"].join(", ") %>)</pre>
+              <pre><%= covering_index[:name] %> (<%= covering_index[:columns].join(", ") %>)</pre>
             </td>
           </tr>
         <% end %>
@@ -270,11 +312,13 @@ remove_index <%= query["unneeded_index"]["table"].to_sym.inspect %>, name: <%= q
     <div id="migration3" style="display: none;">
       <pre>rails g migration add_suggested_indexes</pre>
       <p>And paste</p>
-      <pre style="overflow: scroll; white-space: pre; word-break: normal;"><% @suggested_indexes.each do |index| %>
-<% if index[:using] == "gist" %>
+      <pre style="overflow: scroll; white-space: pre; word-break: normal;">commit_db_transaction
+<% @suggested_indexes.each do |index| %>
+<% if index[:using] && index[:using] != "btree" %>
 connection.execute("CREATE INDEX CONCURRENTLY ON <%= index[:table] %><% if index[:using] %> USING <%= index[:using] %><% end %> (<%= index[:columns].join(", ") %>)")
 <% else %>
-add_index <%= index[:table].to_sym.inspect %>, [<%= index[:columns].map(&:to_sym).map(&:inspect).join(", ") %>], algorithm: :concurrently<% end %><% end %></pre>
+add_index <%= index[:table].to_sym.inspect %>, [<%= index[:columns].map(&:to_sym).map(&:inspect).join(", ") %>], algorithm: :concurrently<% end %>
+<% end %></pre>
     </div>
 
     <% @suggested_indexes.each_with_index do |index, i| %>
@@ -286,33 +330,6 @@ add_index <%= index[:table].to_sym.inspect %>, [<%= index[:columns].map(&:to_sym
   </div>
 <% end %>
 
-<% if @missing_indexes.any? %>
-  <div class="content">
-    <h1>Missing Indexes</h1>
-
-    <p>These tables have a large number of rows but indexes are not used often. Add indexes for faster queries.</p>
-
-    <table class="table">
-      <thead>
-        <tr>
-          <th>Table</th>
-          <th>% of Time Index Used</th>
-          <th>Rows in Table</th>
-        </tr>
-      </thead>
-      <tbody>
-        <% @missing_indexes.each do |query| %>
-          <tr>
-            <td><%= query["table"] %></td>
-            <td style="width: 30%;"><%= query["percent_of_times_index_used"] %></td>
-            <td style="width: 20%;"><%= number_with_delimiter(query["rows_in_table"]) %></td>
-          </tr>
-        <% end %>
-      </tbody>
-    </table>
-  </div>
-<% end %>
-
 <% if !@query_stats_enabled %>
   <div class="content">
     <h1>Query Stats</h1>
@@ -341,3 +358,45 @@ pg_stat_statements.track = all</pre>
     <%= render partial: "queries_table", locals: {queries: @slow_queries} %>
   </div>
 <% end %>
+
+<% if @extended && @unused_indexes.any? %>
+  <div class="content">
+    <h1>Unused Indexes</h1>
+
+    <p>
+      Unused indexes cause unnecessary overhead. Remove them
+      <% if @show_migrations %>
+        <a href="javascript: void(0);" onclick="document.getElementById('migration').style.display = 'block';">with a migration</a>
+      <% end %>
+      for faster writes.
+    </p>
+
+    <div id="migration" style="display: none;">
+      <pre>rails g migration remove_unused_indexes</pre>
+      <p>And paste</p>
+      <pre style="overflow: scroll; white-space: pre; word-break: normal;"><% @unused_indexes.each do |query| %>
+remove_index <%= query[:table].to_sym.inspect %>, name: <%= query[:index].to_s.inspect %><% end %></pre>
+    </div>
+
+    <table class="table">
+      <thead>
+        <tr>
+          <th>Name</th>
+          <th style="width: 20%;">Index Size</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% @unused_indexes.each do |query| %>
+          <tr>
+            <td><%= query[:index] %><div class="text-muted">on <%= query[:table] %></div></td>
+            <td><%= query[:size] %></td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table>
+  </div>
+<% end %>
+
+<script>
+  highlightQueries();
+</script>

data/app/views/pg_hero/home/index_bloat.html.erb

@@ -0,0 +1,69 @@
+<div class="content">
+  <h1>Index Bloat</h1>
+
+  <% if @index_bloat.any? %>
+
+    <p>Indexes can become <%= link_to "bloated over time", "https://www.compose.com/articles/postgresql-bloat-origins-monitoring-and-managing/", target: "_blank" %>. Recreate them to remove bloat.</p>
+
+    <p>For each index, run:</p>
+    <pre><code>CREATE INDEX CONCURRENTLY new_index ...;
+ANALYZE table;
+DROP INDEX CONCURRENTLY index;
+ANALYZE table;
+ALTER INDEX new_index RENAME TO index;</code></pre>
+
+    <p>
+      <% if @show_sql %>
+        <%= link_to "Hide SQL", {} %>
+      <% else %>
+        <%= link_to "Show SQL", {sql: "t"} %>
+      <% end %>
+      for each index
+    </p>
+
+    <table class="table">
+      <thead>
+        <tr>
+          <th>Index</th>
+          <th style="width: 15%;">Bloat</th>
+          <th style="width: 15%;">Size</th>
+        </tr>
+      </thead>
+      <tbody>
+        <% @index_bloat.each do |index| %>
+          <tr>
+            <td>
+              <span style="word-break: break-all;"><%= index[:index] %></span>
+            </td>
+            <td><%= PgHero.pretty_size(index[:bloat_bytes]) %></td>
+            <td><%= PgHero.pretty_size(index[:index_bytes]) %></td>
+          </tr>
+          <% if @show_sql %>
+            <tr>
+              <td colspan="3" style="border-top: none; padding: 0;">
+                <% new_index = "new_#{index[:index]}".first(63) %>
+                <pre><code><%= index[:definition].sub(" INDEX ", " INDEX CONCURRENTLY \n    ").sub(index[:index], new_index) %>;
+
+ANALYZE <%= pghero_pretty_ident(index[:table], schema: index[:schema]) %>;
+
+DROP INDEX CONCURRENTLY
+    <%= pghero_pretty_ident(index[:index]) %>;
+
+ANALYZE <%= pghero_pretty_ident(index[:table], schema: index[:schema]) %>;
+
+ALTER INDEX <%= pghero_pretty_ident(new_index) %>
+    RENAME TO <%= pghero_pretty_ident(index[:index]) %>;</code></pre>
+              </td>
+            </tr>
+          <% end %>
+        <% end %>
+      </tbody>
+    </table>
+  <% else %>
+    <p>No significant index bloat!</p>
+  <% end %>
+</div>
+
+<script>
+  highlightQueries();
+</script>

data/app/views/pg_hero/home/live_queries.html.erb

@@ -1,9 +1,11 @@
 <div class="content">
   <h1>Live Queries</h1>
 
+  <p><%= pluralize(@running_queries.size, "query") %></p>
+
   <%= render partial: "live_queries_table", locals: {queries: @running_queries} %>
 
   <p><%= button_to "Kill all connections", kill_all_path, class: "btn btn-danger" %></p>
 
-  <p class="text-muted">You may need to restart your Rails server afterwards.</p>
+  <p class="text-muted">You may need to restart your app server afterwards.</p>
 </div>

data/app/views/pg_hero/home/maintenance.html.erb

@@ -13,21 +13,25 @@
       <% @maintenance_info.each do |table| %>
         <tr>
           <td>
-            <%= table["table"] %>
-            <% if table["schema"] != "public" %>
-              <span class="text-muted"><%= table["schema"] %></span>
+            <%= table[:table] %>
+            <% if table[:schema] != "public" %>
+              <span class="text-muted"><%= table[:schema] %></span>
             <% end %>
           </td>
           <td>
-            <% time = [table["last_autovacuum"], table["last_vacuum"]].compact.max %>
+            <% time = [table[:last_autovacuum], table[:last_vacuum]].compact.max %>
             <% if time %>
-              <%= @time_zone.parse(time).strftime("%m/%-e %l:%M %P") %>
+              <%= time.in_time_zone(@time_zone).strftime("%-m/%-e %l:%M %P") %>
+            <% else %>
+              <span class="text-muted">Unknown</span>
             <% end %>
           </td>
           <td>
-            <% time = [table["last_autoanalyze"], table["last_analyze"]].compact.max %>
+            <% time = [table[:last_autoanalyze], table[:last_analyze]].compact.max %>
             <% if time %>
-              <%= @time_zone.parse(time).strftime("%m/%-e %l:%M %P") %>
+              <%= time.in_time_zone(@time_zone).strftime("%-m/%-e %l:%M %P") %>
+            <% else %>
+              <span class="text-muted">Unknown</span>
             <% end %>
           </td>
         </tr>

data/app/views/pg_hero/home/queries.html.erb

@@ -7,6 +7,13 @@
 
   <% if @historical_query_stats_enabled %>
     <%= render partial: "query_stats_slider" %>
+  <% elsif @database.query_stats_table_exists? && (columns = @database.missing_query_stats_columns).any? %>
+    <div style="clear: both;">
+      <p>Add missing columns to re-enable historical query stats.</p>
+      <pre><code><% @database.missing_query_stats_columns.each do |column| %>ALTER TABLE pghero_query_stats ADD COLUMN "<%= column %>" <%= column == "query_hash" ? "bigint" : "text" %>;
+<% end %></code></pre>
+      <p>Then restart the web server.</p>
+    </div>
   <% end %>
 
   <% if @query_stats_enabled %>
@@ -14,6 +21,9 @@
       <div class="alert alert-danger">Cannot understand start or end time.</div>
     <% elsif @query_stats.any? || @historical_query_stats_enabled %>
       <%= render partial: "queries_table", locals: {queries: @query_stats, sort_headers: true} %>
+      <script>
+        highlightQueries();
+      </script>
     <% else %>
       <p>Stats are not available yet. Come back soon!</p>
     <% end %>

data/app/views/pg_hero/home/relation_space.html.erb

@@ -0,0 +1,9 @@
+<div class="content">
+  <h1><%= @relation %></h1>
+
+  <h1>Size <small>MB</small></h1>
+  <div id="chart-1" class="chart" style="margin-bottom: 20px;">Loading...</div>
+  <script>
+    new Chartkick.LineChart("chart-1", <%= json_escape(@chart_data.to_json).html_safe %>, {colors: ["#5bc0de"], legend: false, min: null})
+  </script>
+</div>