Skip to content

Comments

Removed the jsonb option to be able to update the mysql database#3281

Open
Spomsoree wants to merge 1 commit intobolt:6.0from
LulububuSoftwareGmbH:bugfix/mysql-jsonb-fix
Open

Removed the jsonb option to be able to update the mysql database#3281
Spomsoree wants to merge 1 commit intobolt:6.0from
LulububuSoftwareGmbH:bugfix/mysql-jsonb-fix

Conversation

@Spomsoree
Copy link
Contributor

This fixed the problem described here: #1905 (comment)

I'm not able to test if fixtures can be installed with postgres. This may undo the fix made in the Issue above.

@bobdenotter
Copy link
Member

bobdenotter commented Aug 10, 2022

I'm looking into this PR, and i don't know quite enough about Postgres to judge it properly. .

About your comment, for what was breaking:

(sfsu is my alias to update the schema: sf doctrine:schema:update)
I run the following: sfsu --dump-sql && sfsu && sfsu --dump-sql:

To update the schema, I normally use bin/console doctrine:schema:update --force. Is that different than sf ?

@Spomsoree
Copy link
Contributor Author

Thats just a alias for php bin/console from the zsh symfony plugin: https://github.com/ohmyzsh/ohmyzsh/blob/master/plugins/symfony2/symfony2.plugin.zsh

It's exactly the same as php bin/console doctrine:schema:update --force

@Spomsoree
Copy link
Contributor Author

@bobdenotter As far as I know jsonb only improves the speed of the postgres database.

I just setup a postgres database, updated the schema, removed the jsonb option, generated migrations, which did result in an empty migrations file.

I think this should be safe to merge.

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

In an effort to keep our pull request list manageable, we are marking this PR as stale because it has been 60 days without activity. If this PR is still relevant, please let us know by posting a quick comment or update so that it can be assigned and hopefully merged.

@github-actions github-actions bot added the Stale label Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants