Skip to content

GetSchema: Batch/parallel access to underlying mysqld for lower latency#63

Closed
teejae wants to merge 5 commits intomasterfrom
tj-get-schema-parallel
Closed

GetSchema: Batch/parallel access to underlying mysqld for lower latency#63
teejae wants to merge 5 commits intomasterfrom
tj-get-schema-parallel

Conversation

@teejae
Copy link
Copy Markdown

@teejae teejae commented May 31, 2020

  • One-pass access for PrimaryKeyColumns
    • getPrimaryKeyColumns retrieves multiple tables from information_schema.key_column_usage instead of using SHOW INDEX
  • Parallel requests for Column metadata and Create Table schema
  • Filters out tables before doing expensive operations

Signed-off-by: Toliver Jue toliver@planetscale.com

@teejae teejae requested a review from sougou as a code owner May 31, 2020 07:46
@teejae teejae requested review from deepthi and enisoc May 31, 2020 07:53
@teejae
Copy link
Copy Markdown
Author

teejae commented May 31, 2020

FYI, depends on PR vitessio#6243 and vitessio#6242.

Diff currently includes rebase from vitessio#6242.

@teejae teejae force-pushed the tj-get-schema-parallel branch 4 times, most recently from 800d27b to 1a545a9 Compare May 31, 2020 09:39
@teejae teejae requested a review from systay June 1, 2020 08:46
@teejae teejae force-pushed the tj-get-schema-parallel branch from 1a545a9 to 9814eb7 Compare June 1, 2020 10:47
Toliver Jue added 2 commits June 1, 2020 19:48
Signed-off-by: Toliver Jue <toliver@planetscale.com>
Signed-off-by: Toliver Jue <toliver@planetscale.com>
@teejae teejae force-pushed the tj-get-schema-parallel branch from 9814eb7 to 1018d62 Compare June 1, 2020 10:49
@teejae teejae requested a review from morgo as a code owner June 1, 2020 10:49
@teejae teejae changed the base branch from tj-get-schema-ctx to master June 1, 2020 10:49
Signed-off-by: Toliver Jue <toliver@planetscale.com>
@teejae teejae marked this pull request as draft June 1, 2020 18:40
@teejae
Copy link
Copy Markdown
Author

teejae commented Jun 1, 2020

FYI converted to draft to clean up PR.

Toliver Jue added 2 commits June 2, 2020 16:34
Signed-off-by: Toliver Jue <toliver@planetscale.com>
@deepthi
Copy link
Copy Markdown

deepthi commented Aug 20, 2020

Merged upstream.

@deepthi deepthi closed this Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants