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

YSQL: Reduce execution time of OpenTable by table_id #10717

Closed
d-uspenskiy opened this issue Nov 29, 2021 · 0 comments
Closed

YSQL: Reduce execution time of OpenTable by table_id #10717

d-uspenskiy opened this issue Nov 29, 2021 · 0 comments
Assignees
Labels
kind/enhancement This is an enhancement of an existing feature
Projects

Comments

@d-uspenskiy
Copy link
Contributor

d-uspenskiy commented Nov 29, 2021

client's OpenTable function calls 2 remote procedures on master:

  • GetTableSchema
  • GetTableLocations

These functions are called sequentially. It is possible to call them simultaneously in case table_id is known.

This will help to reduce OpenTable execution time from 490-515 ms to 328-335 ms (tested on c5d.large RF=3 cluster with master in different region).

@d-uspenskiy d-uspenskiy added the kind/enhancement This is an enhancement of an existing feature label Nov 29, 2021
@d-uspenskiy d-uspenskiy self-assigned this Nov 29, 2021
@d-uspenskiy d-uspenskiy added this to Backlog in YSQL via automation Nov 29, 2021
@d-uspenskiy d-uspenskiy changed the title [YSQL] Reduce execution time of OpenTable by table_id YSQL: Reduce execution time of OpenTable by table_id Nov 29, 2021
d-uspenskiy added a commit that referenced this issue Dec 15, 2021
Summary:
client's `OpenTable` function calls 2 remote procedures on master:
- GetTableSchema
- GetTableLocations

These functions are called sequentially. It is possible to call them simultaneously in case `table_id` is known.

This will help to reduce `OpenTable` execution time from `490-515` ms to `328-335` ms (tested on AWS `c5d.large` RF=3 cluster. Where t-server was running in `US West` and master in `Asia Pacific` regions).
Actually execution time can be reduced to approximately `160`ms (RTT on distributed cluster) by disabling TCP slow start alghoritm.

Test Plan: Jenkins

Reviewers: alex, dsrinivasan, sergei

Reviewed By: sergei

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D14133
YSQL automation moved this from Backlog to Done Dec 15, 2021
d-uspenskiy added a commit that referenced this issue Dec 16, 2021
Summary: In the previous diff https://phabricator.dev.yugabyte.com/D14133 one of `Format`'s argument has been removed, but format string has not been changed accordingly.

Test Plan: Jankins

Reviewers: alex

Reviewed By: alex

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D14398
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement This is an enhancement of an existing feature
Projects
Archived in project
YSQL
  
Done
Development

No branches or pull requests

1 participant