Performance improvement patch for Rails 2.3.x + Ruby 1.9.2

rubyonrails-core
1

Hello,

   Recently I've started migrating my application (Rails 2.3.11) to Ruby 1.9.2. When I run my performance tests, I found performance degradation when associations are preloaded for many records. After profiling, I made a change which noticeably lessen the pain - the fix is rather simple and I'd appreciate if someone would review it:

   #6726 [PATCH] Speedup associations preloading when multiple options are used in :include (2.3.x branch) - Ruby on Rails - rails

   Thanks,    KIR

2

We're only supporting 3.0.x and up. We'll release 2.3.x for critical security issues only.

I suggest starting the upgrade to 3.0.x and fixing the perf problem there (if it still exists).

Thanks!

3

Wait a minute. I thought rails 2.3 was always advertised as 1.9.2 compatible? If so and just because there is a newer version that is also targeted for 1.9.2, should not mean that 1.9.2 tickets for 2-3-stable should be disregarded? Right? Maybe?

- Ken

4

Recently I’ve started migrating my application (Rails 2.3.11) to

Ruby 1.9.2. When I run my performance tests, I found performance

degradation when associations are preloaded for many records. After

profiling, I made a change which noticeably lessen the pain - the fix

is rather simple and I’d appreciate if someone would review it:

We’re only supporting 3.0.x and up. We’ll release 2.3.x for critical

security issues only.

I suggest starting the upgrade to 3.0.x and fixing the perf problem

there (if it still exists).

This problem doesn’t affect 3.0.x users, I’ve already looked at the code.

Anyway, this patch may help someone looking at the similar problem.

Thanks,

KIR

5

A performance flaw is still a flaw. This should be applied to 2.3.X.