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

MySQL migration script: Remove modification for column that no longer exists in Orleans 7 #8314

Merged
merged 2 commits into from
May 18, 2023

Conversation

david-obee
Copy link
Contributor

@david-obee david-obee commented Feb 10, 2023

Problem:

The provided MySQL persistence DB migration script includes a line left over from the Orleans 3.x version, which is no longer applicable to Orleans 7.

As part of the Orleans 7 changes, the OrleansStorage table no longer contains the PayloadJson or PayloadXml columns; it should now only contain the PayloadBinary column.

The line I have removed would try to make a modification on the PayloadJson column, on versions of MySQL 5.7.8 or greater. Assuming you are running the migration on a blank DB, using the CREATE TABLE statement in this script to create your OrleansStorage table, this column will not exist, and this ALTER TABLE line will therefore fail to modify the column.

I suspect it got missed off because the MySQL conditional version syntax makes it look a lot like the line is commented out, when really it just runs on certain versions of MySQL.

I have not found an equivalent issue in any of the other scripts.

Solution:

The solution is just to remove the line, it is no longer necessary.

Tests:

I have run this script and it behaves as expected.

Note: I have not found any associated Github issues that this links to.

@david-obee
Copy link
Contributor Author

@dotnet-policy-service agree

@ReubenBond ReubenBond merged commit 31b37e0 into dotnet:main May 18, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants