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

fixed changing configuration cause crash with MySQL-5.1 #18

Merged
merged 2 commits into from
Nov 11, 2012
Merged

fixed changing configuration cause crash with MySQL-5.1 #18

merged 2 commits into from
Nov 11, 2012

Conversation

samitani
Copy link
Contributor

@glicht
Copy link
Contributor

glicht commented Nov 10, 2012

Thanks for submitting this.

To reproduce the issue, what distribution of MySQL where you using? Can you provide a download link to the exact distribution.

Much thanks,

Guy

@samitani
Copy link
Contributor Author

@glicht
Copy link
Contributor

glicht commented Nov 11, 2012

This does indeed look like a bug. I am merging the pull request but I plan to change this to do this decision dynamically at runtime as we build one binary which is meant to run on all 5.5.x versions. Thus if our plugin is running in 5.5.19 or lower we will want to use the free and in 5.5.20 and higher we will not use it.

@samitani
Copy link
Contributor Author

I agree with your plan.

@glicht glicht merged commit 5864313 into trellix-enterprise:master Nov 11, 2012
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.

2 participants