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

MySQLConnector Hangs when connecting to MySQL 5.7 for DB Open() #342

Closed
jkryspin opened this issue May 31, 2023 · 2 comments
Closed

MySQLConnector Hangs when connecting to MySQL 5.7 for DB Open() #342

jkryspin opened this issue May 31, 2023 · 2 comments
Milestone

Comments

@jkryspin
Copy link

Describe the bug
Bug is as described: https://mysqlconnector.net/troubleshooting/aurora-freeze/
Right now grate hangs on this connection opening after the "Migrating Scripts" printout

To Reproduce
Run grate against an MySQL Aurora 5.7 machine and the connection will hang on the Open call

Expected behavior
image

Screenshots

Additional context
I think there are a few different solutions.

  1. Add a tighter timeout to the Open() call
  2. Add optional setting (default to false) which appends this to the connection string
  3. Any other ideas @erikbra ?

I'm happy to implement some changes once we arise on the proper solution

@jkryspin
Copy link
Author

jkryspin commented Jun 1, 2023

I just realize that the DBconnection.Open() doesn't support cancellations or timeouts. Looks like there is some discussion about this in the MySQLConnector repo. I'm going to close this, but maybe someone will see this post in the future if they need help, the solution is to add Pipelining=False to the connection string

@erikbra
Copy link
Owner

erikbra commented Jun 3, 2023

Thanks for the comment about disabling pipelining, @John-Kryspin . We have the same issues with many of the databases trying to be clever with multiple data streams at once, with multiplexing, pipelining, pooling. But they don't work very well with grate, and we don't benefit much from them.

I have added turning off pipelining by default, if not explicitly enabled in the connection string, here: #346

@erikbra erikbra added this to the 1.5.1 milestone Jun 3, 2023
erikbra added a commit that referenced this issue Jun 3, 2023
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

No branches or pull requests

2 participants