Avoid N+1 problem when loading roles #51

Merged
merged 1 commit into from Mar 13, 2012

Conversation

Projects
None yet
2 participants
@coreyward
Contributor

coreyward commented Mar 10, 2012

Comparing authorizable prompts ActiveRecord to load it, each time individually. Eager loading could be employed, but I think an ID check is sufficient.

Avoid N+1 problem when loading roles
Comparing `authorizable` prompts ActiveRecord to load it, each time individually. Eager loading could be employed, but I think an ID check is sufficient.

be9 added a commit that referenced this pull request Mar 13, 2012

Merge pull request #51 from coreyward/patch-1
Avoid N+1 problem when loading roles

@be9 be9 merged commit 782de8c into be9:master Mar 13, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment