On Feb 1, 2008 10:57 AM, Alexander Dymo <[EMAIL PROTECTED]> wrote: > > Hi, > I've submitted the patch to speedup AR::Base::attributes_with_quotes (and > thus, AR::Base::create and AR::Base::update). > > The idea is that #attributes_with_quotes clones attributes which not only > takes some time, but also allocates quite some memory. > > My measurements show that my rails application during one of lengthy requests > spends 7 extra seconds and allocates (by malloc's) 650 megabytes of memory in > #attributes_with_quotes call (the blog post explaining in details how the > memory consumption was measured and which request tested is at: > http://blog.pluron.com/2008/01/guerrillas-way.html). Of course that memory is > freed by GC, but that takes quite some time to do. > > Could somebody take a look at that patch? Is it safe enough to not clone > attributes in this case?
Being the genius I am, I couldn't find your ticket so I transcribed your guerilla monkey patch to edge rails. I went to link up this very topic in the ticket and finally saw that the patch number was in the subject. Both patches seem to work fine. This is great work, and I'm personally all for it... -- Rick Olson http://lighthouseapp.com http://weblog.techno-weenie.net http://mephistoblog.com --~--~---------~--~----~------------~-------~--~----~ 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 rubyonrails-core@googlegroups.com 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 -~----------~----~----~----~------~----~------~--~---
