Skip to content

Conversation

@mariash
Copy link
Member

@mariash mariash commented Oct 18, 2021

Thank you for submitting a pull request to the diego-release repository. We appreciate the contribution. To help us with getting better context for the pull request please follow these guidelines:

Please make sure to complete the following steps

  • Before PR Submission, Submit an issue for either an Enhancement or Bug
  • Check the Contributing document on how to sign the CLA and run tests in diego-release.
  • Make sure a pull request is done against the develop branch.

Issue Link

Please link the issue you created above for a bug or enhancement.

Thank you!

- In 8ba0b96 we started using a fork of the Go MySQL Driver in order to
  address an issue seen when rolling database nodes, more background in
  this story:
  https://www.pivotaltracker.com/story/show/147786839
- Later versions of the driver include connection liveness checking
  which appears to mitigate the original issue:
  https://github.com/go-sql-driver/mysql/blob/v1.6.0/CHANGELOG.md#version-15-2020-01-07
- The ER_CONNECTION_KILLED error is a MariaDB extension to the protocol,
  we were only able to reproduce the original issue on MariaDB
- Later versions of MariaDB appear to contain a change for compatibility
  with clients that can't handle a sequence number of zero:
  https://jira.mariadb.org/browse/MDEV-19893
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@mariash mariash merged commit 120be3a into develop Oct 21, 2021
@geofffranks geofffranks deleted the pr/179808571-bump-go-mysql-driver branch March 16, 2022 14:23
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