-
Notifications
You must be signed in to change notification settings - Fork 15
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
Optimize select with known bucket_id #252
Optimize select with known bucket_id #252
Conversation
ae0885c
to
4e14f6a
Compare
21420dd
to
10733cb
Compare
4e14f6a
to
a26dbea
Compare
10733cb
to
859a624
Compare
a26dbea
to
2e36906
Compare
The implemented optimization looks okay, but I had idea to find more headrooms in the scope of the issue. Are you plan to do so? |
I thought the plan was only to solve this one and investigate code later as part of some another ticket/epic. |
efae10e
to
920523e
Compare
949bd39
to
fc2e693
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
fc249e9
to
3c53164
Compare
fc2e693
to
6c0f17a
Compare
After this patch, select and pairs requests will no longer fetch sharding key info and extract sharding key info if bucket_id specified. Since calls with specified bucket_id already ignore sharding key values, behavior will not change. Other crud operations already have this optimization. Based on test runs on HP ProBook 440 G7 i7/16Gb, performance had increased by 6-7%. Part of #234
6c0f17a
to
7146f59
Compare
Merge after #251.
After this patch, select and pairs requests will no longer fetch
sharding key info and extract sharding key info if bucket_id specified.
Since calls with specified bucket_id already ignore sharding key
values, behavior will not change. Other crud operations already have
this optimization.
Based on test runs on HP ProBook 440 G7 i7/16Gb, performance had
increased by 6-7%.
I didn't forget about
Part of #234