Feature #6019
closedRevision r34582 significantly slows down rails app
Description
Before r34582 simple rails32 app (query sqlite and disply 20 rows of simple data)
benchmarked as 55req/second , after its performance is 33req/second.
I could agree that RubyOnRails should be in sync with ruby, but maybe there is other way
to fix respond_to?
?
Updated by funny_falcon (Yura Sokolov) about 12 years ago
Fix were relatively easy. Close this issue, please. Excuse me for disturbing.
Updated by naruse (Yui NARUSE) about 12 years ago
- Status changed from Open to Assigned
- Assignee set to knu (Akinori MUSHA)
Updated by funny_falcon (Yura Sokolov) about 12 years ago
Rails already include fix against this issue into its trunk: https://github.com/rails/rails/pull/5039
Any way, could second parameter have more variants? So that:
-
respond_to?(:meth)
andrespond_to?(:meth, nil)
will test for public methods, -
respond_to?(:meth, true)
will test all methods -
respond_to?(:meth, false)
(orrespond_to?(:meth, :with_protected)
) will test for public and protected methods
It is just a suggestion and I could not prove it will be very useful.
Updated by knu (Akinori MUSHA) about 11 years ago
- Tracker changed from Bug to Feature
Updated by knu (Akinori MUSHA) about 11 years ago
- Status changed from Assigned to Closed
- Target version deleted (
2.0.0)
Thanks for the followup. This change was meant to be a fix for a misfeature, and it's great Rails followed it quickly.
As for a feature request, I think we need a little more consideration and details. Please submit one again when you work out a concrete specification with some confidence.