config processing executing code before config processing is done

This appears to be a strange coincidence that when config[:reconnect]
was moved from https://github.com/jruby/activerecord-jdbc-adapter/blob/64157d6e5dfcd64471249d797f87dc1bd255a373/lib/arjdbc/mysql/connection_methods.rb#L65
it lost the local variable 'reconnect' and it changed to calling the
method 'reconnect'.  Creating this method early would cause default
mysql config to crash since the @connection_parameters was not yet
defined.

Author: enebo

lib/arjdbc/mysql/adapter_hash_config.rb

@@ -61,7 +61,8 @@ def build_properties(config)
 
       properties["connectionCollation"] ||= config[:collation] if config[:collation]
 
-      properties["autoReconnect"] ||= reconnect.to_s unless config[:reconnect].nil?
+      reconnect = config[:reconnect]
+      properties["autoReconnect"] ||= reconnect.to_s unless reconnect.nil?
 
       properties["noDatetimeStringSync"] = true unless properties.key?("noDatetimeStringSync")