Skip to content
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

Adapt keyset predicate to be easier for DBMS to optimize #419

Closed
beikov opened this issue May 24, 2017 · 1 comment
Closed

Adapt keyset predicate to be easier for DBMS to optimize #419

beikov opened this issue May 24, 2017 · 1 comment

Comments

@beikov
Copy link
Member

@beikov beikov commented May 24, 2017

Markus Winand explains this very nicely in his article that although the current form for keyset predicates that is implemented might be easier understandable, DBMS might not see through it that easily. Since DBMS can only use top level conjuncts for index lookups, we should rewrite the predicate generation to that form so we get more efficient execution plans.

@beikov beikov added this to the 1.2.0 milestone May 24, 2017
@beikov beikov self-assigned this May 24, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 19, 2017
@beikov beikov added this to In Progress in 1.2.0.Alpha4 Aug 20, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 21, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 22, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 23, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 23, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 23, 2017
Mobe91 added a commit to Mobe91/blaze-persistence that referenced this issue Aug 23, 2017
Mobe91 added a commit that referenced this issue Aug 23, 2017
@Mobe91
Copy link
Contributor

@Mobe91 Mobe91 commented Aug 23, 2017

fixed via a48439d

Loading

@Mobe91 Mobe91 closed this Aug 23, 2017
@beikov beikov moved this from In Progress to Done in 1.2.0.Alpha4 Aug 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants