pghero 1.7.0 → 2.0.0

data/lib/pghero/version.rb

@@ -1,3 +1,3 @@
 module PgHero
-  VERSION = "1.7.0"
+  VERSION = "2.0.0"
 end

data/lib/tasks/pghero.rake

@@ -11,7 +11,7 @@ namespace :pghero do
 
   desc "analyze tables"
   task analyze: :environment do
-    PgHero.analyze_all
+    PgHero.analyze_all(verbose: true, min_size: ENV["MIN_SIZE_GB"].to_f.gigabytes)
   end
 
   desc "autoindex"

data/test/basic_test.rb

@@ -0,0 +1,38 @@
+require_relative "test_helper"
+
+class BasicTest < Minitest::Test
+  def setup
+    City.delete_all
+  end
+
+  def test_explain
+    City.create!
+    PgHero.explain("ANALYZE DELETE FROM cities")
+    assert_equal 1, City.count
+  end
+
+  def test_explain_multiple_statements
+    City.create!
+    assert_raises(ActiveRecord::StatementInvalid) { PgHero.explain("ANALYZE DELETE FROM cities; DELETE FROM cities; COMMIT") }
+  end
+
+  def test_analyze_tables
+    assert PgHero.analyze_tables
+  end
+
+  def test_relation_sizes
+    assert PgHero.relation_sizes
+  end
+
+  def test_transaction_id_danger
+    assert PgHero.transaction_id_danger(threshold: 10000000000)
+  end
+
+  def test_autovacuum_danger
+    assert PgHero.autovacuum_danger
+  end
+
+  def test_databases
+    assert PgHero.databases[:primary].running_queries
+  end
+end

data/test/best_index_test.rb

@@ -11,9 +11,9 @@ class BestIndexTest < Minitest::Test
       found: true,
       structure: {table: "users", where: [{column: "login_attempts", op: "="}], sort: [{column: "created_at", direction: "asc"}]},
       index: {table: "users", columns: ["login_attempts", "created_at"]},
-      rows: 10000,
-      row_estimates: {"login_attempts (=)" => 333, "created_at (sort)" => 1},
-      row_progression: [10000, 333, 0]
+      rows: 5000,
+      row_estimates: {"login_attempts (=)" => 167, "created_at (sort)" => 1},
+      row_progression: [5000, 167, 0]
     }
     assert_equal expected, index
   end

data/test/suggested_indexes_test.rb

@@ -2,9 +2,7 @@ require_relative "test_helper"
 
 class SuggestedIndexesTest < Minitest::Test
   def setup
-    # no pg_stat_statements
     skip if ENV["TRAVIS_CI"]
-
     PgHero.reset_query_stats
   end
 

data/test/test_helper.rb

@@ -10,10 +10,27 @@ Minitest::Test = MiniTest::Unit::TestCase unless defined?(Minitest::Test)
 
 ActiveRecord::Base.establish_connection adapter: "postgresql", database: "pghero_test"
 
+ActiveRecord::Migration.enable_extension "pg_stat_statements"
+
 ActiveRecord::Migration.create_table :cities, force: true do |t|
   t.string :name
 end
 
+ActiveRecord::Migration.create_table :states, force: true do |t|
+  t.string :name
+end
+
+ActiveRecord::Migration.create_table :users, force: true do |t|
+  t.integer :city_id
+  t.integer :login_attempts
+  t.string :email
+  t.string :zip_code
+  t.boolean :active
+  t.timestamp :created_at
+  t.timestamp :updated_at
+end
+ActiveRecord::Migration.add_index :users, :updated_at
+
 class City < ActiveRecord::Base
 end
 
@@ -23,46 +40,27 @@ end
 class User < ActiveRecord::Base
 end
 
-unless ENV["SKIP_SEED"]
-  ActiveRecord::Migration.create_table :users, force: true do |t|
-    t.integer :city_id
-    t.integer :login_attempts
-    t.string :email
-    t.string :zip_code
-    t.boolean :active
-    t.timestamp :created_at
-    t.timestamp :updated_at
+states =
+  50.times.map do |i|
+    {
+      name: "State #{i}"
+    }
   end
-  ActiveRecord::Migration.add_index :users, :updated_at
+State.import states, validate: false
+ActiveRecord::Base.connection.execute("ANALYZE states")
 
-  User.transaction do
-    users =
-      10000.times.map do |i|
-        city_id = i % 100
-        User.new(
-          city_id: city_id,
-          email: "person#{i}@example.org",
-          login_attempts: rand(30),
-          zip_code: i % 40 == 0 ? nil : "12345",
-          active: true,
-          created_at: Time.now - rand(50).days,
-          updated_at: Time.now - rand(50).days
-        )
-      end
-    User.import users, validate: false
+users =
+  5000.times.map do |i|
+    city_id = i % 100
+    {
+      city_id: city_id,
+      email: "person#{i}@example.org",
+      login_attempts: rand(30),
+      zip_code: i % 40 == 0 ? nil : "12345",
+      active: true,
+      created_at: Time.now - rand(50).days,
+      updated_at: Time.now - rand(50).days
+    }
   end
-  ActiveRecord::Base.connection.execute("VACUUM ANALYZE users")
-
-  ActiveRecord::Migration.create_table :states, force: true do |t|
-    t.string :name
-  end
-
-  State.transaction do
-    states =
-      50.times.map do |i|
-        State.new(name: "State #{i}")
-      end
-    State.import states, validate: false
-  end
-  ActiveRecord::Base.connection.execute("VACUUM ANALYZE states")
-end
+User.import users, validate: false
+ActiveRecord::Base.connection.execute("ANALYZE users")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pghero
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Andrew Kane
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-01 00:00:00.000000000 Z
+date: 2017-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -125,11 +125,14 @@ files:
 - app/assets/javascripts/pghero/Chart.bundle.js
 - app/assets/javascripts/pghero/application.js
 - app/assets/javascripts/pghero/chartkick.js
+- app/assets/javascripts/pghero/highlight.pack.js
 - app/assets/javascripts/pghero/jquery.js
 - app/assets/javascripts/pghero/jquery.nouislider.min.js
 - app/assets/stylesheets/pghero/application.css
+- app/assets/stylesheets/pghero/arduino-light.css
 - app/assets/stylesheets/pghero/jquery.nouislider.css
 - app/controllers/pg_hero/home_controller.rb
+- app/helpers/pg_hero/base_helper.rb
 - app/views/layouts/pg_hero/application.html.erb
 - app/views/pg_hero/home/_connections_table.html.erb
 - app/views/pg_hero/home/_live_queries_table.html.erb
@@ -139,10 +142,12 @@ files:
 - app/views/pg_hero/home/connections.html.erb
 - app/views/pg_hero/home/explain.html.erb
 - app/views/pg_hero/home/index.html.erb
-- app/views/pg_hero/home/index_usage.html.erb
+- app/views/pg_hero/home/index_bloat.html.erb
 - app/views/pg_hero/home/live_queries.html.erb
 - app/views/pg_hero/home/maintenance.html.erb
 - app/views/pg_hero/home/queries.html.erb
+- app/views/pg_hero/home/relation_space.html.erb
+- app/views/pg_hero/home/show_query.html.erb
 - app/views/pg_hero/home/space.html.erb
 - app/views/pg_hero/home/system.html.erb
 - app/views/pg_hero/home/tune.html.erb
@@ -171,7 +176,7 @@ files:
 - lib/pghero/methods/maintenance.rb
 - lib/pghero/methods/queries.rb
 - lib/pghero/methods/query_stats.rb
-- lib/pghero/methods/replica.rb
+- lib/pghero/methods/replication.rb
 - lib/pghero/methods/sequences.rb
 - lib/pghero/methods/space.rb
 - lib/pghero/methods/suggested_indexes.rb
@@ -182,8 +187,8 @@ files:
 - lib/pghero/version.rb
 - lib/tasks/pghero.rake
 - pghero.gemspec
+- test/basic_test.rb
 - test/best_index_test.rb
-- test/explain_test.rb
 - test/gemfiles/activerecord41.gemfile
 - test/gemfiles/activerecord42.gemfile
 - test/suggested_indexes_test.rb
@@ -219,8 +224,8 @@ test_files:
 - guides/Query-Stats.md
 - guides/Rails.md
 - guides/Suggested-Indexes.md
+- test/basic_test.rb
 - test/best_index_test.rb
-- test/explain_test.rb
 - test/gemfiles/activerecord41.gemfile
 - test/gemfiles/activerecord42.gemfile
 - test/suggested_indexes_test.rb

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

@@ -1,27 +0,0 @@
-<div class="content">
-  <h1>Index Usage</h1>
-
-  <table class="table">
-    <thead>
-      <tr>
-        <th>Table</th>
-        <th style="width: 30%;">% of Time Index Used</th>
-        <th style="width: 20%;">Rows</th>
-      </tr>
-    </thead>
-    <tbody>
-      <% @index_usage.each do |query| %>
-        <tr>
-          <td>
-            <%= query["table"] %>
-            <% if query["schema"] != "public" %>
-              <span class="text-muted"><%= query["schema"] %></span>
-            <% end %>
-          </td>
-          <td><%= query["percent_of_times_index_used"] %></td>
-          <td><%= number_with_delimiter(query["rows_in_table"]) %></td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-</div>

data/test/explain_test.rb

@@ -1,18 +0,0 @@
-require_relative "test_helper"
-
-class ExplainTest < Minitest::Test
-  def setup
-    City.delete_all
-  end
-
-  def test_explain
-    City.create!
-    PgHero.explain("ANALYZE DELETE FROM cities")
-    assert_equal 1, City.count
-  end
-
-  def test_explain_multiple_statements
-    City.create!
-    assert_raises(ActiveRecord::StatementInvalid) { PgHero.explain("ANALYZE DELETE FROM cities; DELETE FROM cities; COMMIT") }
-  end
-end