acoustid / mbdata

MusicBrainz SQLAlchemy Models
MIT License
97 stars 23 forks source link

2020-05 updates #15

Closed mjpieters closed 4 years ago

mjpieters commented 4 years ago

This brings mbdata up to par with MB Server again.

mjpieters commented 4 years ago

The updates/20200518-pg12-after-upgrade.sql script probably needs some custom re-writing.

I had to run the extensions alterations by hand as a superuser, and alter the role name in two other statements.

sfussenegger commented 4 years ago

Thanks for your effort!

The updates/20200518-pg12-after-upgrade.sql script probably needs some custom re-writing.

I had to run the extensions alterations by hand as a superuser, and alter the role name in two other statements.

Could you please elaborate on this?

mjpieters commented 4 years ago

I had to run the extensions alterations by hand as a superuser, and alter the role name in two other statements.

The before- and after-upgrade scripts appear to have two purposes:

The commands altering or creating extensions need superuser access, so can't be run directly with mbslave. In addition, there are two lines that assign ownership for two functions, and these hardcode the owner role. In our setup we use a different role name so I had to alter those statements accordingly.

dltacube commented 4 years ago

Hey @lalinsky , any reason for delaying this pull request? I'm thinking of using this branch before my upgrade :)

dltacube commented 4 years ago

Thanks @lalinsky