Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Include a preload joins vs. no joins discussion in the preload docs
I'm not sure if Ecto wants to provide this level of guidance in the docs, so I understand if this PR won't be merged. I've just seen many well-meaning people do this wrong (imo) by defaulting to joining all associations just to preload them, resulting in a lot more code and potentially worse performance. I also vaguely remember a tweet from José calling this out, but couldn't find it again. Here I tried to summarize what I think I know about the topic, hoping that it's accurate and helpful.
- Loading branch information