Use :pluck

[sturgill]

Rails 3.2 introduced the pluck method. This prevents you from having to load up full AR model instances just to grab the id field.

[mceachen]

Thanks, but I want to keep Rails 3.0 and 3.1 support, at least for now.

If you want to add a version-aware switch (only do pluck if it's available), that would be great!

[sturgill]

Whether or not this is worth incorporating (given backwards compatibility to pre-pluck days) will depend on the average size of the found sets. Significant gains can happen when there are lots of records as AR is bypassed whenever possible (so those running Rails 3.2.x). But if the average returned set is only a handful of records, this is probably near worthless.

I promise not to be offended if you don't want to incorporate this yet... ;)

[mceachen]

I merged your diff with .try—but .try doesn't shield you from missing methods, only from nil receivers.

.respond_to? is what you wanted.

Thanks for the pluck!

[sturgill]

Yes, you're right - got a little fast with the fingers.

[coveralls]

Changes Unknown when pulling 490a91a on sturgill:master into * on mceachen:master*.