[Suggestion] Add support for eager loading optionally about globalid HOT 4 CLOSED

Can you explain how that would be used and why?

from globalid.

Hello @rafaelfranca
ActiveRecord supports eager load (using includes method). This is useful in some cases that you know you will be calling a lot the relationships of a model, so you end having an N+1 queries issue.

E.g.

users = User.find([ 1, 2 ]) # This generates a query SELECT * FROM users WHERE id IN (1, 2)
users.first.posts # This generates a query SELECT * FROM posts WHERE user_id = 1
users.first.posts # This generates a query SELECT * FROM posts WHERE user_id = 1 again.

In the other hand:

users = User.includes(:posts).find([ 1, 2 ]) # This generates both queries (SELECT * FROM users WHERE id IN (1, 2)) and (SELECT * FROM posts WHERE user_id IN (1, 2))
users.first.posts # No query
users.last.posts # No query 

If we had this feature here, we could do something like GlobalID::Locator.locate_many([ user_gid_1, user_gid_2 ], {includes: :posts}) which would eager load the posts.

I know that can be done with a custom Locator (which I already did for my case) but I thought that having that option could be useful to anyone as it is very generic.

from globalid.

Makes sense. Do you want to open a PR for it?

from globalid.

Sure. Thanks! @rafaelfranca

from globalid.