Merge pull request #4 from planningcenter/bm/dont-retry-query-canceled

mcfadden · web-flow · commit 402640273a96 · 2024-01-29T15:43:15.000-05:00
Fix: Dont retry on ActiveRecord::QueryCanceled
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 1.0.1 (Jan 25, 2024)
+
+- Fix: Do not retry on ActiveRecord::QueryCanceled errors
+
 ## 1.0.0 (Aug 24, 2023)
 
 - Feature: Add support for Rails 7
diff --git a/lib/activerecord/mysql/reconnect.rb b/lib/activerecord/mysql/reconnect.rb
@@ -32,7 +32,8 @@ module Activerecord::Mysql::Reconnect
   ]
 
   DO_NOT_HANDLE_ERROR = [
-    ActiveRecord::StatementTimeout
+    ActiveRecord::StatementTimeout,
+    ActiveRecord::QueryCanceled
   ]
 
   @@handle_r_error_messages = {
diff --git a/lib/activerecord/mysql/reconnect/version.rb b/lib/activerecord/mysql/reconnect/version.rb
@@ -1,7 +1,7 @@
 module Activerecord
   module Mysql
     module Reconnect
-      VERSION = '1.0.0'
+      VERSION = '1.0.1'
     end
   end
 end

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,8 @@ module Activerecord::Mysql::Reconnect`
`32`	`32`	`]`
`33`	`33`
`34`	`34`	`DO_NOT_HANDLE_ERROR = [`
`35`		`- ActiveRecord::StatementTimeout`
	`35`	`+ ActiveRecord::StatementTimeout,`
	`36`	`+ ActiveRecord::QueryCanceled`
`36`	`37`	`]`
`37`	`38`
`38`	`39`	`@@handle_r_error_messages = {`