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

slack-15.0: revert backport of vitessio/vitess#13856 (from #377) #395

Closed
wants to merge 2 commits into from

Conversation

timvaillancourt
Copy link
Member

@timvaillancourt timvaillancourt commented Jun 4, 2024

Description

As an experiment, this PR reverts the backport of vitessio#13856 which was introduced in #377. This PR is in future Vitess versions so this revert cannot be permanent

This backport is suspected of introducing these errors:

Fatal MySQL error: DB-vitess: ERUnknownError 1105 target: pool1.-80.primary: current keyspace is being resharded

Related Issue(s)

  1. slack-15.0: backport vtgate buffer fix vitessio/vitess#13856 #377
  2. VTGate Buffering: Use a more accurate heuristic for determining if we're doing a reshard vitessio/vitess#13856

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

@timvaillancourt timvaillancourt added upstream-revert An upstream revert v15 labels Jun 4, 2024
@timvaillancourt timvaillancourt marked this pull request as ready for review June 4, 2024 15:32
@timvaillancourt timvaillancourt requested a review from a team as a code owner June 4, 2024 15:32
@github-actions github-actions bot added this to the v15.0.5 milestone Jun 4, 2024
@timvaillancourt
Copy link
Member Author

As discussed with @tanjinx, this cannot be the cause of the issues we're seeing as we are not running v15 vtgates in the environment we are troubleshooting 👍

@timvaillancourt timvaillancourt deleted the revert-bp-pr13856-slack-15.0 branch June 4, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream-revert An upstream revert v15
Projects
None yet
2 participants