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.

Sign up for GitHub

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

Jump to bottom

slack-19.0: Ensure all topo read calls consider --topo_read_concurrency #571

Open
wants to merge 6 commits into
base: slack-19.0
Choose a base branch
from
Open

slack-19.0: Ensure all topo read calls consider --topo_read_concurrency #571

wants to merge 6 commits into from

Conversation

timvaillancourt
Copy link
Member

@timvaillancourt timvaillancourt commented Dec 4, 2024 •
edited
Loading

Description

This PR is an early backport of PRs to address VTOrc efficiency:

  1. Ensure all topo read calls consider --topo_read_concurrency vitessio/vitess#17276

Related Issue(s)

Checklist

  • "Backport to:" labels have been added if this change should be back-ported
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on the CI
  • Documentation was added or is not required

Deployment Notes

@github-actions github-actions bot added this to the v19.0.7 milestone Dec 4, 2024
@timvaillancourt
semaphore per cell, update help+changelog
a0b15be 
Signed-off-by: Tim Vaillancourt <[email protected]>
@timvaillancourt
help typo
a5051cf 
Signed-off-by: Tim Vaillancourt <[email protected]>
@timvaillancourt
better stat name
eb4a624 
Signed-off-by: Tim Vaillancourt <[email protected]>
@timvaillancourt
better stat name again
50b95e1 
Signed-off-by: Tim Vaillancourt <[email protected]>
@timvaillancourt timvaillancourt marked this pull request as ready for review December 10, 2024 18:36
@timvaillancourt timvaillancourt requested a review from a team as a code owner December 10, 2024 18:36
@timvaillancourt timvaillancourt changed the title Ensure all topo read calls consider --topo_read_concurrency slack-19.0: Ensure all topo read calls consider --topo_read_concurrency Dec 10, 2024
@timvaillancourt
reset op start time
48a72eb 
Signed-off-by: Tim Vaillancourt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Component: VTCtl Component: VTGate Component: VTOrc Component: VTTablet upstream-backport An upstream backport v19 v22-backport vtorc
Projects
None yet
Milestone
v19.0.7
Development

Successfully merging this pull request may close these issues.
1 participant
@timvaillancourt