Skip to content

Instantly share code, notes, and snippets.

@mauricioszabo
Forked from defunkt/connection_fix.rb
Created October 10, 2011 18:38
Show Gist options
  • Star 4 You must be signed in to star a gist
  • Fork 4 You must be signed in to fork a gist
  • Save mauricioszabo/1276131 to your computer and use it in GitHub Desktop.
Save mauricioszabo/1276131 to your computer and use it in GitHub Desktop.
MySQL server has gone away fix
# If your workers are inactive for a long period of time, they'll lose
# their MySQL connection.
#
# This hack ensures we re-connect whenever a connection is
# lost. Because, really. why not?
#
# Stick this in RAILS_ROOT/config/initializers/connection_fix.rb (or somewhere similar)
#
# From:
# http://coderrr.wordpress.com/2009/01/08/activerecord-threading-issues-and-resolutions/
#Prevents Rails from loading multiple times. I could use a global variable too, but still...
if !$LOADED_FEATURES.include?(__FILE__)
if ActiveRecord::ConnectionAdapters.const_defined?(:MysqlAdapter)
module ActiveRecord::ConnectionAdapters
class MysqlAdapter
alias_method :execute_without_retry, :execute
def execute(*args)
execute_without_retry(*args)
rescue ActiveRecord::StatementInvalid => e
if e.message =~ /server has gone away/i
warn "Server timed out, retrying"
reconnect!
retry
else
raise e
end
end
end
end
elsif ActiveRecord::ConnectionAdapters.const_defined?(:Mysql2Adapter)
module ActiveRecord::ConnectionAdapters
class Mysql2Adapter
alias_method :execute_without_retry, :execute
def execute(*args)
execute_without_retry(*args)
rescue ActiveRecord::StatementInvalid => e
if e.message =~ /server has gone away/i
warn "Server timed out, retrying"
reconnect!
retry
else
raise e
end
end
end
end
end
end
#Prevents Rails from loading multiple times. I could use a global variable too, but still...
$LOADED_FEATURES << __FILE__
@microwaves
Copy link

Thank you for the fixes!

@ctrochalakis
Copy link

you probably mean "if not"

if ! $LOADED_FEATURES.include? __FILE__

@rapimo
Copy link

rapimo commented Jan 23, 2012

Thanks a lot for this fix!
However
Line 13 should be

if !$LOADED_FEATURES.include?(__FILE__)

@mauricioszabo
Copy link
Author

Already corrected. Thanks :)

@rapimo
Copy link

rapimo commented Jan 24, 2012

I was actually talking about the missing ? question mark on include?

@mauricioszabo
Copy link
Author

Ok, thanks again :) Corrected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment