Skip to content

Comments

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

Closed
timvaillancourt wants to merge 12 commits intoslack-19.0from
bp-pr17276.slack-19.0
Closed

slack-19.0: Ensure all topo read calls consider --topo_read_concurrency#571
timvaillancourt wants to merge 12 commits intoslack-19.0from
bp-pr17276.slack-19.0

Conversation

@timvaillancourt
Copy link

@timvaillancourt timvaillancourt commented Dec 4, 2024

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

Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@github-actions github-actions bot added this to the v19.0.7 milestone Dec 4, 2024
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@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
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>
@timvaillancourt timvaillancourt deleted the bp-pr17276.slack-19.0 branch January 13, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant