<resending w/ a more enticing subject for core folks>
[EMAIL PROTECTED] wrote:
> I believe clear_reloadable_connections! should be as follows:
> 
> # Clears the cache which maps classes
>  def clear_reloadable_connections!
>   @@active_connections.each do |name, conn|
>    if conn.supports_reloading?
>       conn.disconnect!
>       @@active_connections.delete(name)
>    end
>  end
> end

Agreed. I don't fully understand the intent of the reloadable bit, from
the comments on the initial checkin it seems related to a sqlite issue. 
I don't understand why the current code would _not_ disconnect the 
connection (if it doesn't support reloading), but would still delete it 
from the cache.

Can someone from core comment?

And agree that this seems like a blocker on 1.2.



--~--~---------~--~----~------------~-------~--~----~
 You received this message because you are subscribed to the Google Groups 
"Ruby on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to