diff options
author | Sean Griffin <sean@thoughtbot.com> | 2014-06-12 09:00:03 -0600 |
---|---|---|
committer | Sean Griffin <sean@thoughtbot.com> | 2014-06-12 17:23:23 -0600 |
commit | d82dafe65c1137a2f84975c202d471054533f7bc (patch) | |
tree | ad493c4ce897d58a97b79276221d42d7837e1e18 /actionpack | |
parent | 46139d33c06715e74ad450428ece3ee84da98579 (diff) | |
download | rails-d82dafe65c1137a2f84975c202d471054533f7bc.tar.gz rails-d82dafe65c1137a2f84975c202d471054533f7bc.tar.bz2 rails-d82dafe65c1137a2f84975c202d471054533f7bc.zip |
Fix performance regression on preloading HABTM associations
We'd spend a lot of time calling `hash` and `eql?` on the join model,
which has no primary key. Calling `id` with no primary key is a really
slow way to get back `nil`, so we can improve the performance there.
However, even with the escape clause, we *still* weren't getting high
enough performance, as we were checking the primary key too much. `hash`
will always return `nil.hash` for records with no id, and `==` will
always return `false`. We can optimize those cases in the HABTM join
model.
Diffstat (limited to 'actionpack')
0 files changed, 0 insertions, 0 deletions