pgdice 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9232acb1dfa33aad924a95525d3af64784e8ac525d100e1f90b58708d5d61e7b
-  data.tar.gz: e5d06ae456620ff0f2c807e274dc33a65ad893087e10b8f3ad6616947f688464
+  metadata.gz: 6ae586abaf672ced69ab9bab60300d99827b9b39363077c1ead5376c74823ba3
+  data.tar.gz: 5cd703d8edaaf0315e62a761ea1a34d7f8b9e07b7a9e5caa09505d16ce20030e
 SHA512:
-  metadata.gz: 4262c4066a0d4eaf1305875b06a90cd71570dd4eb1e0da8dfc934080aa1f04ed47c084548df4cd3c39a55f7c726fba7a15cafd4804ada6228217922fb4e36509
-  data.tar.gz: 2df09b63917ef19edac7bcc4a5a61724f0bf3221572403e8684bb808908ea749993f8a5d9348a615b2eb2a4c282d2a909a7e0dd600919834b6e0d59bd2b3c0fa
+  metadata.gz: 93ebdada6b6e423bb1b0befda9bae91b52d3e19b498c818d38dac31268f77b21462b75b17bfc4c95281915e38c786a8f16cc2297fb97090b1c2756730c4c8706
+  data.tar.gz: 8c25ece5a48b2972b0571de6be7a0a6912c193af0ad3193224b1946bf70b04f1771b6d209ccdb7e720aeeb9b53f4ae3ab74542de1bae22d33a17f11dd8e3c00b

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [v0.4.2] : 2019-04-22
+### Changes
+  - Fix #19 where PgDice wouldn't recover from a broken PG::Connection 
+  by adding new retry behavior
+
 ## [v0.4.1] : 2019-03-21
 ### Changes
   - Fix bug where partitioning by months would break when the month was < 10

data/lib/pgdice.rb

@@ -42,6 +42,8 @@ require 'pgdice/partition_manager_factory'
 require 'pgdice/partition_dropper'
 require 'pgdice/partition_dropper_factory'
 
+require 'pgdice/query_executor'
+
 require 'pgdice/database_connection'
 require 'pgdice/database_connection_factory'
 

data/lib/pgdice/database_connection_factory.rb

@@ -10,7 +10,8 @@ module PgDice
 
     def initialize(configuration, opts = {})
       @configuration = configuration
-      @query_executor = opts[:query_executor] ||= ->(query) { pg_connection.exec(query) }
+      @query_executor = opts[:query_executor] ||= PgDice::QueryExecutor.new(logger: logger,
+                                                                            connection_supplier: -> { pg_connection })
     end
 
     def call

data/lib/pgdice/query_executor.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+# Entry point for DatabaseConnection
+module PgDice
+  # Wrapper class around pg_connection to reset connection on PG errors
+  class QueryExecutor
+    attr_reader :logger
+
+    def initialize(logger:, connection_supplier:)
+      @logger = logger
+      @connection_supplier = connection_supplier
+    end
+
+    def call(query)
+      @connection_supplier.call.exec(query)
+    rescue PG::Error => error
+      logger.error { "Caught error: #{error}. Going to reset connection and try again" }
+      @connection_supplier.call.reset
+      @connection_supplier.call.exec(query)
+    end
+  end
+end

data/lib/pgdice/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgDice
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pgdice
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Andrew Newell
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-03-21 00:00:00.000000000 Z
+date: 2019-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg
@@ -325,6 +325,7 @@ files:
 - lib/pgdice/period_fetcher_factory.rb
 - lib/pgdice/pg_slice_manager.rb
 - lib/pgdice/pg_slice_manager_factory.rb
+- lib/pgdice/query_executor.rb
 - lib/pgdice/table.rb
 - lib/pgdice/table_finder.rb
 - lib/pgdice/validation.rb