viewy 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d0d346f0a7be5e2986dfc28c44ce149b6fc47f5d
-  data.tar.gz: c771b0de1a80db6c793766f11a5ffb6ec32fc511
+  metadata.gz: 5eff5c5e346be1ca5ab408f4b37431d8af0b102a
+  data.tar.gz: 4699cfb5e13e4331e455f6e970afceb566050e30
 SHA512:
-  metadata.gz: 778cd9146f3d4be1c9f8ed9954b3316cc67e0ef6ee8df7adcc234dbf4820a0a23774310c74d818b0f7e304b0c2189b5b23000248b4c319eb5ca5f999e29e2781
-  data.tar.gz: 5d0a92306ab08af4e2725894f40fa5de6c3a032ff66931b3f700e15b7e17f0a8958bedf7002804723f81c42f92e53a8f0b61ed212e9fbb15d6de29db59cd2c91
+  metadata.gz: 763d4f573e981231058164708e19507e82e904093a7f095cde7aabf27e3dd335e381dba3a2d7e6701bc31ddf4d5362e0dedd10c59cb8bfe145ae0cc6d6e2401a
+  data.tar.gz: 673a0b105715622bc7a8966a777f7855073ce9d5a42daab3ca4780757a6ccec6636e64c9fd482c0a9b07361df2390c5f82e755cced92eeecb08c86cc2b44916e

data/db/migrate/20160513141153_add_index_recreation.rb

@@ -72,12 +72,16 @@ class AddIndexRecreation < ActiveRecord::Migration
             FOREACH current_statement IN ARRAY create_statements LOOP
               EXECUTE current_statement;
             END LOOP;
-            FOREACH current_index_statement IN ARRAY index_statements LOOP
-              EXECUTE current_index_statement;
-            END LOOP;
-            FOREACH current_index_statement IN ARRAY current_index_statements LOOP
-              EXECUTE current_index_statement;
-            END LOOP;
+            IF index_statements IS NOT NULL THEN
+              FOREACH current_index_statement IN ARRAY index_statements LOOP
+                EXECUTE current_index_statement;
+              END LOOP;
+            END IF;
+            IF current_index_statements IS NOT NULL THEN
+              FOREACH current_index_statement IN ARRAY current_index_statements LOOP
+                EXECUTE current_index_statement;
+              END LOOP;
+            END IF;
             ALTER EVENT TRIGGER view_dependencies_update ENABLE ALWAYS;
           END;
       $$ LANGUAGE plpgsql;

data/lib/viewy/dependency_management/view_refresher.rb

@@ -42,7 +42,29 @@ module Viewy
       end
 
       private def refresh_single_view(view_name, concurrently:)
-        connection.execute(refresh_sql(view_name, concurrently))
+        begin
+          connection.execute(refresh_sql(view_name, concurrently))
+        rescue ActiveRecord::StatementInvalid => ex
+          if attempt_non_concurrent_refresh?(ex, concurrently)
+            connection.execute(refresh_sql(view_name, false))
+          end
+        end
+      end
+
+      # @param ex [ActiveRecord::StatementInvalid] an exception raised during the view refresh
+      # @param concurrently [Boolean] whether or not the refresh that raised the exception was concurrent
+      #
+      # @raise [ActiveRecord::StatementInvalid] re-raised if the original exception is not a FeatureNotSupported exception
+      #
+      # @return [Boolean] true if the system should attempt to refresh the view non-concurrently after a concurrent
+      #   refresh has failed.  Since Postgres raises an error when an unpopulated view is refreshed concurrently, this
+      #   allows the system to populated such views if needed
+      private def attempt_non_concurrent_refresh?(ex, concurrently)
+        if ex.original_exception.instance_of?(PG::FeatureNotSupported)
+          concurrently
+        else
+          raise ex
+        end
       end
 
       # @param name [String] the name of a materialized view

data/lib/viewy/version.rb

@@ -1,3 +1,3 @@
 module Viewy
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: viewy
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Emerson Huitt
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-13 00:00:00.000000000 Z
+date: 2016-05-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails