New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include performance optimizations #1333

Merged
merged 3 commits into from Feb 2, 2014

Conversation

1 participant
@mickhansen
Contributor

mickhansen commented Feb 2, 2014

This pull request optimizes include performance for 1:1 associations and associations to models where there is only one primary key

(this goes for most associations, although auto generated through models on N:M associations will still suffer the old slowdown - need to refactor primary key support for these first, 2.0 stuff)

@mickhansen mickhansen referenced this pull request Feb 2, 2014

Closed

Sequelize performance #1265

mickhansen added a commit that referenced this pull request Feb 2, 2014

Merge pull request #1333 from sequelize/include-performance-optimizat…
…ions

Include performance optimizations

@mickhansen mickhansen merged commit de3a736 into master Feb 2, 2014

1 check passed

default The Travis CI build passed
Details

@mickhansen mickhansen deleted the include-performance-optimizations branch Feb 2, 2014

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